Messages in this thread Patch in this message | | | From | Li kunyu <> | Subject | [PATCH] ceph/decode: Remove unnecessary ‘ 0’ values from ret | Date | Sun, 3 Sep 2023 04:11:12 +0800 |
| |
ret is assigned first, so it does not need to initialize the assignment. Bad is not used and can be removed.
Signed-off-by: Li kunyu <kunyu@nfschina.com> --- net/ceph/decode.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/net/ceph/decode.c b/net/ceph/decode.c index bc109a1a4616..9f5f095d8235 100644 --- a/net/ceph/decode.c +++ b/net/ceph/decode.c @@ -50,7 +50,7 @@ static int ceph_decode_entity_addr_legacy(void **p, void *end, struct ceph_entity_addr *addr) { - int ret = -EINVAL; + int ret = 0; /* Skip rest of type field */ ceph_decode_skip_n(p, end, 3, bad); @@ -66,8 +66,7 @@ ceph_decode_entity_addr_legacy(void **p, void *end, sizeof(addr->in_addr), bad); addr->in_addr.ss_family = be16_to_cpu((__force __be16)addr->in_addr.ss_family); - ret = 0; -bad: + return ret; } -- 2.18.2
| |