diff options
| author | Denis Vlasenko <vda.linux@googlemail.com> | 2007-01-29 22:51:44 +0000 |
|---|---|---|
| committer | Denis Vlasenko <vda.linux@googlemail.com> | 2007-01-29 22:51:44 +0000 |
| commit | 06c0a71d2315756db874e98bc4f760ca3283b6a6 (patch) | |
| tree | df385c84041f3fd8328e7a50caef4495ef2734a8 /miscutils | |
| parent | b6aae0f38194cd39960a898606ee65d4be93a895 (diff) | |
| download | busybox-w32-06c0a71d2315756db874e98bc4f760ca3283b6a6.tar.gz busybox-w32-06c0a71d2315756db874e98bc4f760ca3283b6a6.tar.bz2 busybox-w32-06c0a71d2315756db874e98bc4f760ca3283b6a6.zip | |
preparatory patch for -Wwrite-strings #3
Diffstat (limited to 'miscutils')
| -rw-r--r-- | miscutils/crontab.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/miscutils/crontab.c b/miscutils/crontab.c index d442272f0..eb7c2059c 100644 --- a/miscutils/crontab.c +++ b/miscutils/crontab.c | |||
| @@ -290,8 +290,7 @@ static void EditFile(const char *user, const char *file) | |||
| 290 | /* | 290 | /* |
| 291 | * CHILD - change user and run editor | 291 | * CHILD - change user and run editor |
| 292 | */ | 292 | */ |
| 293 | char *ptr; | 293 | const char *ptr; |
| 294 | char visual[1024]; | ||
| 295 | 294 | ||
| 296 | if (ChangeUser(user, 1) < 0) | 295 | if (ChangeUser(user, 1) < 0) |
| 297 | exit(0); | 296 | exit(0); |
| @@ -299,10 +298,9 @@ static void EditFile(const char *user, const char *file) | |||
| 299 | if (ptr == NULL || strlen(ptr) > 256) | 298 | if (ptr == NULL || strlen(ptr) > 256) |
| 300 | ptr = PATH_VI; | 299 | ptr = PATH_VI; |
| 301 | 300 | ||
| 302 | snprintf(visual, sizeof(visual), "%s %s", ptr, file); | 301 | ptr = xasprintf("%s %s", ptr, file); |
| 303 | execl(DEFAULT_SHELL, DEFAULT_SHELL, "-c", visual, NULL); | 302 | execl(DEFAULT_SHELL, DEFAULT_SHELL, "-c", ptr, NULL); |
| 304 | perror("exec"); | 303 | bb_perror_msg_and_die("exec"); |
| 305 | exit(0); | ||
| 306 | } | 304 | } |
| 307 | if (pid < 0) { | 305 | if (pid < 0) { |
| 308 | /* | 306 | /* |