From c238a97a9e90734da15163ba54d6862bb8e360e9 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Mon, 25 Apr 2005 05:24:35 +0000 Subject: use asprintf in place of malloc/sprintf as suggested by solar --- e2fsprogs/lsattr.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) (limited to 'e2fsprogs/lsattr.c') diff --git a/e2fsprogs/lsattr.c b/e2fsprogs/lsattr.c index be5723c43..d3896637d 100644 --- a/e2fsprogs/lsattr.c +++ b/e2fsprogs/lsattr.c @@ -101,29 +101,31 @@ static int lsattr_dir_proc(const char *dir_name, struct dirent *de, { STRUCT_STAT st; char *path; - int dir_len = strlen(dir_name); + int i = strlen(dir_name); - path = malloc(dir_len + strlen(de->d_name) + 2); - - if (dir_len && dir_name[dir_len-1] == '/') - sprintf(path, "%s%s", dir_name, de->d_name); + if (i && dir_name[i-1] == '/') + i = asprintf(&path, "%s%s", dir_name, de->d_name); else - sprintf(path, "%s/%s", dir_name, de->d_name); + i = asprintf(&path, "%s/%s", dir_name, de->d_name); + if (i == -1) + bb_perror_msg_and_die("asprintf failed"); + if (LSTAT(path, &st) == -1) bb_perror_msg(path); else { if (de->d_name[0] != '.' || (flags & OPT_ALL)) { list_attributes(path); if (S_ISDIR(st.st_mode) && (flags & OPT_RECUR) && - strcmp(de->d_name, ".") && - strcmp(de->d_name, "..")) { + strcmp(de->d_name, ".") && strcmp(de->d_name, "..")) { printf("\n%s:\n", path); iterate_on_dir(path, lsattr_dir_proc, NULL); printf("\n"); } } } + free(path); + return 0; } -- cgit v1.2.3