diff options
| author | kib <kib@FreeBSD.org> | 2012-04-28 18:57:27 +0000 |
|---|---|---|
| committer | kib <kib@FreeBSD.org> | 2012-04-28 18:57:27 +0000 |
| commit | 3378e556f13cad9cb91bd3cfee7d6a9147202305 (patch) | |
| tree | 52470c4e1134ce892fbc5c9b04bc5b30b0c35415 /lib/libc/rpc/svc_raw.c | |
| parent | f67b4d433308479a900040e4c5f3e54133015eed (diff) | |
| download | FreeBSD-src-3378e556f13cad9cb91bd3cfee7d6a9147202305.zip FreeBSD-src-3378e556f13cad9cb91bd3cfee7d6a9147202305.tar.gz | |
Fix several memory and lock leaks on the out of memory condition.
Reported by: Matt Miller <matt matthewjmiller net>
MFC after: 1 week
Diffstat (limited to 'lib/libc/rpc/svc_raw.c')
| -rw-r--r-- | lib/libc/rpc/svc_raw.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/lib/libc/rpc/svc_raw.c b/lib/libc/rpc/svc_raw.c index 67bcba1..de95152 100644 --- a/lib/libc/rpc/svc_raw.c +++ b/lib/libc/rpc/svc_raw.c @@ -96,10 +96,22 @@ svc_raw_create() mutex_unlock(&svcraw_lock); return (NULL); } - if (__rpc_rawcombuf == NULL) + if (__rpc_rawcombuf == NULL) { __rpc_rawcombuf = calloc(UDPMSGSIZE, sizeof (char)); + if (__rpc_rawcombuf == NULL) { + free(srp); + mutex_unlock(&svcraw_lock); + return (NULL); + } + } srp->raw_buf = __rpc_rawcombuf; /* Share it with the client */ srp->server = svc_xprt_alloc(); + if (srp->server == NULL) { + free(__rpc_rawcombuf); + free(srp); + mutex_unlock(&svcraw_lock); + return (NULL); + } svc_raw_private = srp; } srp->server->xp_fd = FD_SETSIZE; |
