Date: Wed, 1 Aug 2001 03:25:13 -0700 (PDT) From: Ian Dowse <iedowse@FreeBSD.org> To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/nfs nfs_vnops.c Message-ID: <200108011025.f71APD623218@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
iedowse 2001/08/01 03:25:13 PDT
Modified files:
sys/nfs nfs_vnops.c
Log:
Fix a client-side memory leak in nfs_flush(). The code allocates
a temporary array to store struct buf pointers if the list doesn't
fit in a local array. Usually it frees the array when finished,
but if it jumps to the 'again' label and the new list does fit in
the local array then it can forget to free a previously malloc'd
M_TEMP memory.
Move the free() up a line so that it frees any previously allocated
memory whether or not it needs to malloc a new array.
Reviewed by: dillon
Revision Changes Path
1.172 +3 -3 src/sys/nfs/nfs_vnops.c
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200108011025.f71APD623218>