diff options
| author | Mike Frysinger | 2005-04-25 05:24:35 +0000 |
|---|---|---|
| committer | Mike Frysinger | 2005-04-25 05:24:35 +0000 |
| commit | c238a97a9e90734da15163ba54d6862bb8e360e9 (patch) | |
| tree | 5f7aa8b3884f523eea53f35e69a77362ca352235 /e2fsprogs/chattr.c | |
| parent | 7eb6457c4cf76ace4e738c01911c78ec52e51840 (diff) | |
| download | busybox-c238a97a9e90734da15163ba54d6862bb8e360e9.zip busybox-c238a97a9e90734da15163ba54d6862bb8e360e9.tar.gz | |
use asprintf in place of malloc/sprintf as suggested by solar
Diffstat (limited to 'e2fsprogs/chattr.c')
| -rw-r--r-- | e2fsprogs/chattr.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/e2fsprogs/chattr.c b/e2fsprogs/chattr.c index 5aaa14f..07cee8f 100644 --- a/e2fsprogs/chattr.c +++ b/e2fsprogs/chattr.c @@ -167,11 +167,9 @@ static int chattr_dir_proc(const char *dir_name, struct dirent *de, /*if (strcmp(de->d_name, ".") && strcmp(de->d_name, "..")) {*/ if (de->d_name[0] == '.' && (de->d_name[1] == '\0' || \ (de->d_name[1] == '.' && de->d_name[2] == '\0'))) { - - char *path = malloc(strlen(dir_name) + 1 + strlen(de->d_name) + 1); - if (!path) - bb_error_msg_and_die("Couldn't allocate path variable in chattr_dir_proc"); - sprintf(path, "%s/%s", dir_name, de->d_name); + char *path; + if (asprintf(&path, "%s/%s", dir_name, de->d_name) == -1) + bb_error_msg_and_die("asprintf failed"); change_attributes(path); free(path); } |