summaryrefslogtreecommitdiffstats
path: root/api
diff options
context:
space:
mode:
authorJose A. Rivera <jarrpa@redhat.com>2014-01-24 09:56:17 -0600
committerAnand Avati <avati@redhat.com>2014-01-24 23:46:35 -0800
commit8814fdafab377139cdf3863096234f4d5b6e543e (patch)
treec9ae5232d5ed8477cd198e44174954ea5f8cc1e3 /api
parent8a21f10ad373dad117557555a7db43d489d05b28 (diff)
libgfapi: Fix pointer dereference before NULL check
ctx is not checked for NULL before it is dereferenced, though the check in "err" seems to suggest we should guard against it. BUG: 789278 CID: 1124827 Change-Id: I6ed0b42992af5f370cce9aa2feb56811cdd54b42 Signed-off-by: Jose A. Rivera <jarrpa@redhat.com> Reviewed-on: http://review.gluster.org/6782 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'api')
-rw-r--r--api/src/glfs.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/api/src/glfs.c b/api/src/glfs.c
index 1bae78d23..2b4312eac 100644
--- a/api/src/glfs.c
+++ b/api/src/glfs.c
@@ -68,6 +68,10 @@ glusterfs_ctx_defaults_init (glusterfs_ctx_t *ctx)
xlator_mem_acct_init (THIS, glfs_mt_end + 1);
+ if (!ctx) {
+ goto err;
+ }
+
ctx->process_uuid = generate_glusterfs_ctx_id ();
if (!ctx->process_uuid) {
goto err;