trouble with strndup function

I keep getting the following warning with this code:

Quote:

---

warning: assignment makes pointer from integer without a cast

---

Referring to the line containing the strndup function call. The strndup function is copying the string but it is copying the whole string.

I am not sure what I am doing wrong, help would be great.

Code:

---

          char *inet;
          char *str;
          char  *field_delimeter;
          char received[BUFSIZ] = "blah blah ( copy me ) blah";
 
          // Get delimeters
          str = strchr( received, '(' );
          field_delimeter =  strrchr( received, ')' );
 
          // Get new string
          if (( inet = strndup( str, field_delimeter - str))== NULL){ // this line returns warning
                  printf( "error duping string\n" );
                  return EXIT_FAILURE;
          }

---

Did you provide the correct prototype for the strndup function? Did you #include the right file?

yes, I did (string.h), sorry for not including in my code.

I just realised it is copying the correct nr of chars I was printing the wrong variable in my test code... that was an hour well wasted :P

But I am still getting that warning? Dont know why??

Normally the error means the prototype hasn't been provided, so the compiler assumes strndup returns an int, therefore you get an error when converting said int to a pointer.

strndup is none standard. Are you sure its in string.h (you'll have to open it up and check for yourself).

from sting.h

Code:

---

/* Return a malloc'd copy of at most N bytes of STRING.  The
  resultant string is terminated even if no null terminator
  appears before STRING[N].  */
#if defined __USE_GNU
extern char *strndup (__const char *__string, size_t __n)
    __THROW __attribute_malloc__;
#endif

---

Is there anything else that could cause the warning?

Yes, strndup isn't an ANSI function, so if you specified -ansi on the command line, all the GNU extensions (of which this is one) are no longer visible to you due to the conditional compilation.

Either stop using -ansi (because your program is no longer ANSI-compatible), or write your own implementation of strndup() in terms of ANSI functions.

I am not using -ansi but I will do my own implementation to solve the problem and go cross platform.

Thanks Salem