what's wrong with this strtok_r()

[g]

hello!

I get a segm fault when I use strtok_r function

the code :

#ifndef CLIENT_MSG_H_
#define CLIENT_MSG_H_
#include <string>
class Client_msg
{
public:
Client_msg();

~Client_msg();

const char* data() const
{
return data_;
}

char* data()
{
return data_;
}

std::string get_user();
std::string get_service();
std::auto_ptr<s td::stringget_a rguments();

private:
char data_[2048];
char **last;
};

#endif /*CLIENT_MSG_H_*/



#include "Client_msg .h"

Client_msg::Cli ent_msg()
{
}

Client_msg::~Cl ient_msg()
{
}

std::string Client_msg::get _user()
{
std::string user(strtok_r(d ata_,"#",last)) ;
return user;
}

std::string Client_msg::get _service()
{
std::string service(strtok_ r(data_,"#",las t));
return service;
}

std::auto_ptr<s td::stringClien t_msg::get_argu ments()
{
std::auto_ptr<s td::stringargum ents(new
std::string(str tok_r(data_,"#" ,last)));
return arguments;
}
any idea ??

thanks in advance,

[Victor Bazarov]

g wrote:

hello!

I get a segm fault when I use strtok_r function

the code :

[...]

>

any idea ??

Well, a couple... First off, there is no standard 'strtok_r'
function. Second, there are three instance of calling 'strtok_r'
in your code, which one is failing? Third, your code does not
have the 'main' function so I can't compile it and run it to see
the alleged behaviour (even if my compiler did provide the
'strtok_r' function).

Now, could it be that the arguments (like the first one) that
you try to provide to 'strtok_r' is somehow wrong? Like, not
initialised, maybe...

V
--
Please remove capital 'A's when replying by e-mail
I do not respond to top-posted replies, please don't ask

[Larry Smith]

g wrote:

hello!

I get a segm fault when I use strtok_r function

the code :

#ifndef CLIENT_MSG_H_
#define CLIENT_MSG_H_
#include <string>
class Client_msg
{
public:
Client_msg();

~Client_msg();

const char* data() const
{
return data_;
}

char* data()
{
return data_;
}

std::string get_user();
std::string get_service();
std::auto_ptr<s td::stringget_a rguments();

private:
char data_[2048];
char **last;
};

#endif /*CLIENT_MSG_H_*/



#include "Client_msg .h"

Client_msg::Cli ent_msg()
{
}

Client_msg::~Cl ient_msg()
{
}

std::string Client_msg::get _user()
{
std::string user(strtok_r(d ata_,"#",last)) ;
return user;
}

std::string Client_msg::get _service()
{
std::string service(strtok_ r(data_,"#",las t));
return service;
}

std::auto_ptr<s td::stringClien t_msg::get_argu ments()
{
std::auto_ptr<s td::stringargum ents(new
std::string(str tok_r(data_,"#" ,last)));
return arguments;
}
any idea ??

thanks in advance,

Where is data_ initialized?

See 'man strtok_r'.

I assume you are trying to extract a series
of tokens, seperated by the '#' char, from
data_??

strtok_r is a Posix enhancement to strtok.
Note that 'data_' should be passed only on the first
call, pass NULL on subsequent calls. strtok modifies
it input (data_ in this case). As the man pages says:

"Never use these functions."

Read the strtok/strtok_r man page carefully.

Try using a C++ std::string instead of the
C style array (data_).

[g]

I assume you are trying to extract a series

of tokens, seperated by the '#' char, from
data_??

yes...

strtok_r is a Posix enhancement to strtok.
Note that 'data_' should be passed only on the first
call, pass NULL on subsequent calls. strtok modifies
it input (data_ in this case). As the man pages says:

"Never use these functions."

why????

Try using a C++ std::string instead of the
C style array (data_).

I wish but I have to use char[]. Client_msg use is for getting incoming
data from clients request's and my network lib (Boost.Asio) dont accept
an std::string for read operations! :-(((

so I have to play with C style char []/* to avoid some copies:

std::string request(network _lib.get_data);// where network_lib.get _data
returns a char[]

std::string get_user(std::s ting&request);
std::string get_service(std ::sting&request );
....

thats why I prefer to play with char and C functions instead of just
copy the char[] to a string.

thanks in advance

[Larry Smith]

g wrote:

>I assume you are trying to extract a series
of tokens, seperated by the '#' char, from
data_??

yes...

>strtok_r is a Posix enhancement to strtok.
Note that 'data_' should be passed only on the first
call, pass NULL on subsequent calls. strtok modifies
it input (data_ in this case). As the man pages says:

"Never use these functions."

why????

>Try using a C++ std::string instead of the
C style array (data_).

I wish but I have to use char[]. Client_msg use is for getting incoming
data from clients request's and my network lib (Boost.Asio) dont accept
an std::string for read operations! :-(((

so I have to play with C style char []/* to avoid some copies:

std::string request(network _lib.get_data);// where network_lib.get _data
returns a char[]

std::string get_user(std::s ting&request);
std::string get_service(std ::sting&request );
....

thats why I prefer to play with char and C functions instead of just
copy the char[] to a string.

thanks in advance

Read the strtok 'man' pages carefully, and
ask further questions in one of these newsgroups:

comp.os.linux.d evelopment.syst em
comp.os.linux.d evelopment.apps