diff options
author | Akhil Bhansali <bhansaliakhil@gmail.com> | 2015-09-10 15:56:21 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2015-09-15 02:32:18 -0700 |
commit | 26917cb3bb476e6b2452405baa5698f7fb01ceda (patch) | |
tree | f4246cec28143b9eef2ca47b28f2a046d6579492 | |
parent | 7b9135045685125d7c94d75f06d762fa1c5ba4b9 (diff) |
BZ 789278: Coverity bug fixes for logically dead code
Removing the logically dead code as reported by coverity tool run
on GlusterFS.
The code changes are removal of logically dead code, hence did not
run the testcases.
CIDs Fixed starts from 1292652 to 1292663 in sequence.
Signed-off-by: Akhil Bhansali <bhansaliakhil@gmail.com>
Change-Id: I05b35f744c89b5e49b6322635c7a0d367ef10abb
BUG: 789278
Reviewed-on: http://review.gluster.org/12150
Reviewed-by: Anoop C S <anoopcs@redhat.com>
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
-rw-r--r-- | xlators/features/upcall/src/upcall.c | 29 |
1 files changed, 0 insertions, 29 deletions
diff --git a/xlators/features/upcall/src/upcall.c b/xlators/features/upcall/src/upcall.c index 4954d43db07..fc04d4d5d51 100644 --- a/xlators/features/upcall/src/upcall.c +++ b/xlators/features/upcall/src/upcall.c @@ -79,7 +79,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (open, frame, -1, op_errno, NULL, NULL); return 0; @@ -136,8 +135,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (writev, frame, -1, op_errno, NULL, NULL, NULL); return 0; @@ -197,7 +194,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (readv, frame, -1, op_errno, NULL, 0, NULL, NULL, NULL); @@ -253,7 +249,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (lk, frame, -1, op_errno, NULL, NULL); return 0; @@ -310,7 +305,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (truncate, frame, -1, op_errno, NULL, NULL, NULL); return 0; @@ -373,7 +367,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (setattr, frame, -1, op_errno, NULL, NULL, NULL); return 0; @@ -435,7 +428,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (rename, frame, -1, op_errno, NULL, NULL, NULL, NULL, NULL, NULL); @@ -493,7 +485,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (unlink, frame, -1, op_errno, NULL, NULL, NULL); return 0; @@ -550,7 +541,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (link, frame, -1, op_errno, NULL, NULL, NULL, NULL, NULL); @@ -609,7 +599,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (rmdir, frame, -1, op_errno, NULL, NULL, NULL); return 0; @@ -669,7 +658,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (mkdir, frame, -1, op_errno, NULL, NULL, NULL, NULL, NULL); @@ -733,7 +721,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (create, frame, -1, op_errno, NULL, NULL, NULL, NULL, NULL, NULL); @@ -792,7 +779,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (lookup, frame, -1, op_errno, NULL, NULL, NULL, NULL); @@ -849,7 +835,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (stat, frame, -1, op_errno, NULL, NULL); return 0; @@ -878,7 +863,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (fstat, frame, -1, op_errno, NULL, NULL); return 0; @@ -907,7 +891,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (ftruncate, frame, -1, op_errno, NULL, NULL, NULL); @@ -963,7 +946,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (access, frame, -1, op_errno, NULL); return 0; @@ -1020,7 +1002,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (readlink, frame, -1, op_errno, NULL, NULL, NULL); @@ -1081,7 +1062,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (mknod, frame, -1, op_errno, NULL, NULL, NULL, NULL, NULL); @@ -1143,7 +1123,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (symlink, frame, -1, op_errno, NULL, NULL, NULL, NULL, NULL); @@ -1200,7 +1179,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (opendir, frame, -1, op_errno, NULL, NULL); return 0; @@ -1256,7 +1234,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (statfs, frame, -1, op_errno, NULL, NULL); return 0; @@ -1312,7 +1289,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (readdir, frame, -1, op_errno, NULL, NULL); return 0; @@ -1341,7 +1317,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (readdirp, frame, -1, op_errno, NULL, NULL); return 0; @@ -1370,7 +1345,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (fsetattr, frame, -1, op_errno, NULL, NULL, NULL); @@ -1428,7 +1402,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (fallocate, frame, -1, op_errno, NULL, NULL, NULL); @@ -1486,7 +1459,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (discard, frame, -1, op_errno, NULL, NULL, NULL); @@ -1544,7 +1516,6 @@ out: return 0; err: - op_errno = (op_errno == -1) ? errno : op_errno; UPCALL_STACK_UNWIND (zerofill, frame, -1, op_errno, NULL, NULL, NULL); |