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

The callers of kern_getfsstat(UIO_SYSSPACE) expect that *buf always

Browse Source 
returns memory which must be freed, regardless of the error.  Assign
NULL to *buf in case we are not going to allocate any memory due to
invalid mode.

Reported and tested by:	pho
Reviewed by:	jhb
Sponsored by:	The FreeBSD Foundation
MFC after:	3 weeks (together with r310638)
Differential revision:	https://reviews.freebsd.org/D9042
This commit is contained in:
Konstantin Belousov 2017-01-04 16:09:45 +00:00
parent a94dccbe27
commit 6c4338f2ef
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sys/kern/vfs_syscalls.c
View File

@ -429,6 +429,8 @@ kern_getfsstat(struct thread *td, struct statfs **buf, size_t bufsize,
case MNT_NOWAIT: case MNT_NOWAIT:
break; break;
default: default:
if (bufseg == UIO_SYSSPACE)
*buf = NULL;
return (EINVAL); return (EINVAL);
} }
restart: restart: