diff options
| author | Niels de Vos <ndevos@redhat.com> | 2017-04-10 10:10:30 +0200 |
|---|---|---|
| committer | Jeff Darcy <jeff@pl.atyp.us> | 2017-04-10 11:19:24 -0400 |
| commit | 88551d8567503b3eba17e54fbe7dd9ee6266c1ab (patch) | |
| tree | c01193c0fe77431cd66a89a831b7dbfd60ef9f97 /cli/src/cli-rpc-ops.c | |
| parent | 3594f960a8b5ccf81b6d06c89003f175ddd6a2c8 (diff) | |
upcall: correct the cleanup path in init() when GF_CALLOC() fails
Coverity found that "priv" can be NULL when the 1st GF_CALLOC() fails,
and still is being used with "priv->xattrs" before the functon returns.
While cleaning this up, also removing the log message for out of memory,
as this is already logged through GF_CALLOC().
BUG: 789278
Change-Id: I887eaa9136dc25a39c107cd2152b1ba11f9fa925
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: https://review.gluster.org/17023
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: soumya k <skoduri@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Amar Tumballi <amarts@redhat.com>
Diffstat (limited to 'cli/src/cli-rpc-ops.c')
0 files changed, 0 insertions, 0 deletions