diff options
| author | Ilya Dryomov <idryomov@gmail.com> | 2019-08-30 17:38:31 +0200 |
|---|---|---|
| committer | Ilya Dryomov <idryomov@gmail.com> | 2020-03-30 12:42:41 +0200 |
| commit | bb0e681dda33adac516dd74723b757c5c2feafe6 (patch) | |
| tree | 801194158c65ed87a73743229eae4fc492e5a523 /net | |
| parent | libceph: simplify ceph_monc_handle_map() (diff) | |
| download | linux-bb0e681dda33adac516dd74723b757c5c2feafe6.tar.xz linux-bb0e681dda33adac516dd74723b757c5c2feafe6.zip | |
libceph: directly skip to the end of redirect reply
Coverity complains about a double write to *p. Don't bother with
osd_instructions and directly skip to the end of redirect reply.
Reported-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/ceph/osd_client.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c index f9ec6a6568bd..998e26b75a78 100644 --- a/net/ceph/osd_client.c +++ b/net/ceph/osd_client.c @@ -3483,9 +3483,6 @@ static int ceph_redirect_decode(void **p, void *end, goto e_inval; } - len = ceph_decode_32(p); - *p += len; /* skip osd_instructions */ - /* skip the rest */ *p = struct_end; out: |