diff options
author | Jean-Frederic Clere <jfclere@apache.org> | 2010-02-16 17:07:09 +0000 |
---|---|---|
committer | Jean-Frederic Clere <jfclere@apache.org> | 2010-02-16 17:07:09 +0000 |
commit | f2e461d5592949f5221ff9bad7f3a47df9fb169a (patch) | |
tree | a46af0fa4300a1f42f87fec3c15a8304f2b490aa /shmem | |
parent | a81d5077631eb861de6d40431a94a60314401e8e (diff) | |
download | apr-f2e461d5592949f5221ff9bad7f3a47df9fb169a.tar.gz |
Make sure we don't leak file descriptors.
git-svn-id: https://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); |