converting char** to char* or char

Question

I have a old program in which some library function is used and i dont have that library.

So I am writing that program using libraries of c++. In that old code some function is there which is called like this

*string = newstrdup("Some string goes here");

the string variable is declared as char **string;

What he may be doing in that function named "newstrdup" ? I tried many things but i dont know what he is doing ... Can anyone help

Answer 1

newstrdup is probably making a new string that is a duplicate of the passed string; it returns a pointer to the string (which is itself a pointier to the characters).

Answer 2

It looks like he's written a strdup() function to operate on an existing pointer, probably to re-allocate it to a new size and then fill its contents. Likely, he's doing this to re-use the same pointer in a loop where *string is going to change frequently while preventing a leak on every subsequent call to strdup().

I'd probably implement that like string = redup(&string, "new contents") .. but that's just me.

Edit:

Here's a snip of my 'redup' function which might be doing something similar to what you posted, just in a different way:

int redup(char **s1, const char *s2)
{
    size_t len, size;

    if (s2 == NULL)
     return -1;

    len = strlen(s2);
    size = len + 1;

    *s1 = realloc(*s1, size);

    if (*s1 == NULL)
     return -1;

    memset(*s1, 0, size);
    memcpy(*s1, s2, len);

    return len;
}

Of course, I should probably save a copy of *s1 and restore it if realloc() fails, but I didn't need to get that paranoid.

Answer 3

I think you need to look at what is happening with the "string" variable within the code as the prototype for the newstrdup() function would appear to be identical to the library strdup() version.

Are there any free(*string) calls in the code?

It would appear to be a strange thing do to, unless it's internally keeping a copy of the duplicated string and returning a pointer back to the same string again.

Again, I would ask why?

Answer 4

there has to be a reason that they wrote a "new" version of strdup. So there must be a corner case that it handles differently. like perhaps a null string returns an empty string.

litb's answer is a replacement for strdup, but I would think there is a reason they did what they did.

If you want to use strdup directly, use a define to rename it, rather than write new code.

Answer 5

The line *string = newstrdup("Some string goes here"); is not showing any weirdness to newstrdup. If string has type char ** then newstrdup is just returning char * as expected. Presumably string was already set to point to a variable of type char * in which the result is to be placed. Otherwise the code is writing through an uninitialized pointer..