d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bail out when memory allocation is failed, rather than referencing

Browse Source 
a NULL pointer.

PR:		kern/94480
Submitted by:	Michiel Pelt <m.pelt xs4all nl>
This commit is contained in:
Xin LI 2009-04-02 17:16:39 +00:00
parent 42e607776b
commit 06f535757c
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=190646
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
lib/libufs/block.c
View File

@ -64,8 +64,10 @@ bread(struct uufsd *disk, ufs2_daddr_t blockno, void *data, size_t size)
*/
if (((intptr_t)data) & 0x3f) {
p2 = malloc(size);
if (p2 == NULL)
if (p2 == NULL) {
ERROR(disk, "allocate bounce buffer");
goto fail;
}
}
cnt = pread(disk->d_fd, p2, size, (off_t)(blockno * disk->d_bsize));
if (cnt == -1) {
@ -115,8 +117,10 @@ bwrite(struct uufsd *disk, ufs2_daddr_t blockno, const void *data, size_t size)
*/
if (((intptr_t)data) & 0x3f) {
p2 = malloc(size);
if (p2 == NULL)
if (p2 == NULL) {
ERROR(disk, "allocate bounce buffer");
return (-1);
}
memcpy(p2, data, size);
data = p2;
}