diff options
author | Wayne Davison <wayned@samba.org> | 2006-11-09 02:34:17 +0000 |
---|---|---|
committer | Wayne Davison <wayned@samba.org> | 2006-11-09 02:34:17 +0000 |
commit | 89a0e3a927dff09070ef9454bc9215efaba91036 (patch) | |
tree | 87bb78aaad8c7316283fa6ae6c693a50e2fd510d /popt | |
parent | a8facdc09070ba9d01f67de1b08e2704ca34c440 (diff) | |
download | android_external_rsync-89a0e3a927dff09070ef9454bc9215efaba91036.tar.gz android_external_rsync-89a0e3a927dff09070ef9454bc9215efaba91036.tar.bz2 android_external_rsync-89a0e3a927dff09070ef9454bc9215efaba91036.zip |
Get rid of the last strcpy() call by using an static inline function.
Diffstat (limited to 'popt')
-rw-r--r-- | popt/system.h | 26 |
1 files changed, 17 insertions, 9 deletions
diff --git a/popt/system.h b/popt/system.h index d2bbb9d8..0579b8ca 100644 --- a/popt/system.h +++ b/popt/system.h @@ -95,9 +95,25 @@ char * xstrdup (const char *str) /*@*/; /*@=redecl =redef@*/ +#ifndef HAVE_STRLCPY +size_t strlcpy(char *d, const char *s, size_t bufsize); +#endif + +#ifndef HAVE_STRLCAT +size_t strlcat(char *d, const char *s, size_t bufsize); +#endif + #if HAVE_MCHECK_H && defined(__GNUC__) #define vmefail() (fprintf(stderr, "virtual memory exhausted.\n"), exit(EXIT_FAILURE), NULL) -#define xstrdup(_str) (strcpy((malloc(strlen(_str)+1) ? : vmefail()), (_str))) +static inline char * +xstrdup(const char *s) +{ + size_t memsize = strlen(s) + 1; + char *ptr = malloc(memsize); + if (!ptr) vmefail(); + strlcpy(ptr, s, memsize); + return ptr; +} #else #define xstrdup(_str) strdup(_str) #endif /* HAVE_MCHECK_H && defined(__GNUC__) */ @@ -106,14 +122,6 @@ char * xstrdup (const char *str) #define getenv(_s) __secure_getenv(_s) #endif -#ifndef HAVE_STRLCPY -size_t strlcpy(char *d, const char *s, size_t bufsize); -#endif - -#ifndef HAVE_STRLCAT -size_t strlcat(char *d, const char *s, size_t bufsize); -#endif - #if !defined HAVE_SNPRINTF || !defined HAVE_C99_VSNPRINTF #define snprintf rsync_snprintf int snprintf(char *str,size_t count,const char *fmt,...); |