diff options
| author | davidxu <davidxu@FreeBSD.org> | 2005-12-16 02:50:53 +0000 |
|---|---|---|
| committer | davidxu <davidxu@FreeBSD.org> | 2005-12-16 02:50:53 +0000 |
| commit | f2547368a8bf5177a138b8873c03912af57e90af (patch) | |
| tree | 6ae4ec26d50bc59ff2d9306207a3b957bc977ae6 /lib/libc/stdio/vfscanf.c | |
| parent | 1aa9ee553eae931f7025dd4707c9fd70d8789fbb (diff) | |
| download | FreeBSD-src-f2547368a8bf5177a138b8873c03912af57e90af.zip FreeBSD-src-f2547368a8bf5177a138b8873c03912af57e90af.tar.gz | |
With current pthread implementations, a mutex initialization will
allocate a memory block. sscanf calls __svfscanf which in turn calls
fread, fread triggers mutex initialization but the mutex is not
destroyed in sscanf, this leads to memory leak. To avoid the memory
leak and performance issue, we create a none MT-safe version of fread:
__fread, and instead let __svfscanf call __fread.
PR: threads/90392
Patch submitted by: dhartmei
MFC after: 7 days
Diffstat (limited to 'lib/libc/stdio/vfscanf.c')
| -rw-r--r-- | lib/libc/stdio/vfscanf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/libc/stdio/vfscanf.c b/lib/libc/stdio/vfscanf.c index ea3004c..27df280 100644 --- a/lib/libc/stdio/vfscanf.c +++ b/lib/libc/stdio/vfscanf.c @@ -412,7 +412,7 @@ literal: } nread += sum; } else { - size_t r = fread((void *)va_arg(ap, char *), 1, + size_t r = __fread((void *)va_arg(ap, char *), 1, width, fp); if (r == 0) |
