diff options
| author | Eygene Ryabinkin <rea-fbsd@codelabs.ru> | 2008-08-31 22:17:30 +0400 |
|---|---|---|
| committer | Michael Brown <mcb30@etherboot.org> | 2008-09-04 11:29:37 +0100 |
| commit | 6de45ad4ae65d1754abaee4d5e92851b7b40c9b5 (patch) | |
| tree | 60e06cadb331e70741d118914becb9c85bbc21c4 | |
| parent | 99251f5b326487ef1cd15fb448b20d1c3b534db8 (diff) | |
| download | ipxe-6de45ad4ae65d1754abaee4d5e92851b7b40c9b5.tar.gz | |
[util] Avoid calling fclose(NULL) in zbin.c
Must check that argument to a fclose() is not NULL -- we can get to the
'err' label when file was not opened. fclose(NULL) is known to produce
core dump on some platforms and we don't want zbin to fail so loudly.
Signed-off-by: Eygene Ryabinkin <rea-fbsd@codelabs.ru>
| -rw-r--r-- | src/util/zbin.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/util/zbin.c b/src/util/zbin.c index f47fa36ba..b24f401eb 100644 --- a/src/util/zbin.c +++ b/src/util/zbin.c @@ -90,7 +90,8 @@ static int read_file ( const char *filename, void **buf, size_t *len ) { return 0; err: - fclose ( file ); + if ( file ) + fclose ( file ); return -1; } |