diff options
| author | cperciva <cperciva@FreeBSD.org> | 2004-01-26 15:05:30 +0000 |
|---|---|---|
| committer | cperciva <cperciva@FreeBSD.org> | 2004-01-26 15:05:30 +0000 |
| commit | 3a15ac7b11b935e1782804d428a4147f3f5fec27 (patch) | |
| tree | 98dad340eeda79234192f74dcba37022c19180a8 /sbin/fsck_ffs/pass1.c | |
| parent | 7ce7ecdff6a94c18c2c9e30b1348c73f3e3a6175 (diff) | |
| download | FreeBSD-src-3a15ac7b11b935e1782804d428a4147f3f5fec27.zip FreeBSD-src-3a15ac7b11b935e1782804d428a4147f3f5fec27.tar.gz | |
Avoid dereferencing null pointers in fsck_ffs. (pfatal may return,
so it isn't a safe way of handling [mc]alloc failures.)
PR: misc/61800
Approved by: rwatson (mentor)
Diffstat (limited to 'sbin/fsck_ffs/pass1.c')
| -rw-r--r-- | sbin/fsck_ffs/pass1.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sbin/fsck_ffs/pass1.c b/sbin/fsck_ffs/pass1.c index 272b5f1..341d027 100644 --- a/sbin/fsck_ffs/pass1.c +++ b/sbin/fsck_ffs/pass1.c @@ -141,7 +141,7 @@ pass1(void) } info = calloc((unsigned)inosused, sizeof(struct inostat)); if (info == NULL) - pfatal("cannot alloc %u bytes for inoinfo\n", + errx(EEXIT, "cannot alloc %u bytes for inoinfo", (unsigned)(sizeof(struct inostat) * inosused)); inostathead[c].il_stat = info; /* @@ -175,7 +175,7 @@ pass1(void) } info = calloc((unsigned)inosused, sizeof(struct inostat)); if (info == NULL) - pfatal("cannot alloc %u bytes for inoinfo\n", + errx(EEXIT, "cannot alloc %u bytes for inoinfo", (unsigned)(sizeof(struct inostat) * inosused)); memmove(info, inostathead[c].il_stat, inosused * sizeof(*info)); free(inostathead[c].il_stat); |
