author | Lars Hjemli <hjemli@gmail.com> | 2009-11-07 17:08:30 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2009-11-07 17:08:30 (UTC) |
commit | 21f67e7d82986135922aece6b4ebf410a98705bc (patch) (unidiff) | |
tree | 67256f0ffbac072f1a19147079822d07bfe951ac | |
parent | 8cfe4897f01066ae901bdd6ef106faf8e8f2ddf2 (diff) | |
download | cgit-21f67e7d82986135922aece6b4ebf410a98705bc.zip cgit-21f67e7d82986135922aece6b4ebf410a98705bc.tar.gz cgit-21f67e7d82986135922aece6b4ebf410a98705bc.tar.bz2 |
shared.c: return original errno
Noticed-by: Andreas Schwab <schwab@linux-m68k.org>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | shared.c | 8 |
1 files changed, 5 insertions, 3 deletions
@@ -387,36 +387,38 @@ int cgit_close_filter(struct cgit_filter *filter) | |||
387 | chk_non_negative(dup2(filter->old_stdout, STDOUT_FILENO), | 387 | chk_non_negative(dup2(filter->old_stdout, STDOUT_FILENO), |
388 | "Unable to restore STDOUT"); | 388 | "Unable to restore STDOUT"); |
389 | close(filter->old_stdout); | 389 | close(filter->old_stdout); |
390 | if (filter->pid < 0) | 390 | if (filter->pid < 0) |
391 | return 0; | 391 | return 0; |
392 | waitpid(filter->pid, &filter->exitstatus, 0); | 392 | waitpid(filter->pid, &filter->exitstatus, 0); |
393 | if (WIFEXITED(filter->exitstatus) && !WEXITSTATUS(filter->exitstatus)) | 393 | if (WIFEXITED(filter->exitstatus) && !WEXITSTATUS(filter->exitstatus)) |
394 | return 0; | 394 | return 0; |
395 | die("Subprocess %s exited abnormally", filter->cmd); | 395 | die("Subprocess %s exited abnormally", filter->cmd); |
396 | } | 396 | } |
397 | 397 | ||
398 | /* Read the content of the specified file into a newly allocated buffer, | 398 | /* Read the content of the specified file into a newly allocated buffer, |
399 | * zeroterminate the buffer and return 0 on success, errno otherwise. | 399 | * zeroterminate the buffer and return 0 on success, errno otherwise. |
400 | */ | 400 | */ |
401 | int readfile(const char *path, char **buf, size_t *size) | 401 | int readfile(const char *path, char **buf, size_t *size) |
402 | { | 402 | { |
403 | int fd; | 403 | int fd, e; |
404 | struct stat st; | 404 | struct stat st; |
405 | 405 | ||
406 | fd = open(path, O_RDONLY); | 406 | fd = open(path, O_RDONLY); |
407 | if (fd == -1) | 407 | if (fd == -1) |
408 | return errno; | 408 | return errno; |
409 | if (fstat(fd, &st)) { | 409 | if (fstat(fd, &st)) { |
410 | e = errno; | ||
410 | close(fd); | 411 | close(fd); |
411 | return errno; | 412 | return e; |
412 | } | 413 | } |
413 | if (!S_ISREG(st.st_mode)) { | 414 | if (!S_ISREG(st.st_mode)) { |
414 | close(fd); | 415 | close(fd); |
415 | return EISDIR; | 416 | return EISDIR; |
416 | } | 417 | } |
417 | *buf = xmalloc(st.st_size + 1); | 418 | *buf = xmalloc(st.st_size + 1); |
418 | *size = read_in_full(fd, *buf, st.st_size); | 419 | *size = read_in_full(fd, *buf, st.st_size); |
420 | e = errno; | ||
419 | (*buf)[*size] = '\0'; | 421 | (*buf)[*size] = '\0'; |
420 | close(fd); | 422 | close(fd); |
421 | return (*size == st.st_size ? 0 : errno); | 423 | return (*size == st.st_size ? 0 : e); |
422 | } | 424 | } |