diff options
author | Eitan Adler <eadler@FreeBSD.org> | 2012-12-05 13:56:56 +0000 |
---|---|---|
committer | Eitan Adler <eadler@FreeBSD.org> | 2012-12-05 13:56:56 +0000 |
commit | 5ebc65ad1cb4dbb699e760c6a4c6b5b52efa94e0 (patch) | |
tree | b1fc281f41d512a48f9787a4982b9b47af306431 | |
parent | 3d0fb3a57e72d37356b6a765070b3a2289837f77 (diff) | |
download | pw-darwin-5ebc65ad1cb4dbb699e760c6a4c6b5b52efa94e0.tar.gz pw-darwin-5ebc65ad1cb4dbb699e760c6a4c6b5b52efa94e0.tar.zst pw-darwin-5ebc65ad1cb4dbb699e760c6a4c6b5b52efa94e0.zip |
Simplify string duplication: use strdup instead of malloc + strcpy
Submitted by: db
Approved by: cperciva
MFC after: 2 weeks
-rw-r--r-- | pw/grupd.c | 11 |
1 files changed, 5 insertions, 6 deletions
@@ -50,12 +50,11 @@ setgrdir(const char * dir) { if (dir == NULL) return -1; - else { - char * d = malloc(strlen(dir)+1); - if (d == NULL) - return -1; - grpath = strcpy(d, dir); - } + else + grpath = strdup(dir); + if (grpath == NULL) + return -1; + return 0; } |