diff options
author | jfclere <jfclere@13f79535-47bb-0310-9956-ffa450edef68> | 2010-02-16 17:07:09 +0000 |
---|---|---|
committer | jfclere <jfclere@13f79535-47bb-0310-9956-ffa450edef68> | 2010-02-16 17:07:09 +0000 |
commit | 39806ed8a079c06d4903a8e8cd389e5747987b34 (patch) | |
tree | a46af0fa4300a1f42f87fec3c15a8304f2b490aa /shmem | |
parent | bf1717254700fa80cdf3c37c973c2aed847c3d2f (diff) | |
download | libapr-39806ed8a079c06d4903a8e8cd389e5747987b34.tar.gz |
Make sure we don't leak file descriptors.
git-svn-id: http://svn.apache.org/repos/asf/apr/apr/trunk@910597 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'shmem')
-rw-r--r-- | shmem/unix/shm.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/shmem/unix/shm.c b/shmem/unix/shm.c index 081ca6749..42921955a 100644 --- a/shmem/unix/shm.c +++ b/shmem/unix/shm.c @@ -314,26 +314,31 @@ APR_DECLARE(apr_status_t) apr_shm_create(apr_shm_t **m, * exist before calling ftok(). */ new_m->shmkey = ftok(filename, 1); if (new_m->shmkey == (key_t)-1) { + apr_file_close(file); return errno; } if ((new_m->shmid = shmget(new_m->shmkey, new_m->realsize, SHM_R | SHM_W | IPC_CREAT | IPC_EXCL)) < 0) { + apr_file_close(file); return errno; } if ((new_m->base = shmat(new_m->shmid, NULL, 0)) == (void *)-1) { + apr_file_close(file); return errno; } new_m->usable = new_m->base; if (shmctl(new_m->shmid, IPC_STAT, &shmbuf) == -1) { + apr_file_close(file); return errno; } apr_uid_current(&uid, &gid, pool); shmbuf.shm_perm.uid = uid; shmbuf.shm_perm.gid = gid; if (shmctl(new_m->shmid, IPC_SET, &shmbuf) == -1) { + apr_file_close(file); return errno; } @@ -341,6 +346,7 @@ APR_DECLARE(apr_status_t) apr_shm_create(apr_shm_t **m, status = apr_file_write(file, (const void *)&reqsize, &nbytes); if (status != APR_SUCCESS) { + apr_file_close(file); return status; } status = apr_file_close(file); |