]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
objecter: move map checks to helper
authorSage Weil <sage@inktank.com>
Mon, 22 Oct 2012 04:07:12 +0000 (21:07 -0700)
committerSage Weil <sage@inktank.com>
Mon, 22 Oct 2012 04:07:12 +0000 (21:07 -0700)
commit2d20f3a8a4cd36da71df18c0eb58057273d6af59
tree7f434a69a5e63e2b64f89f6427e49a9785662412
parentaae72c259ca9bee1a21f32608b4e63cc39f07787
objecter: move map checks to helper

This makes coverity happier because we check_op_pool_dne() may free
the Op (or Lingerop) structure(s), but the callers in the submit_*
paths dereference after calling.  This is actually safe because they
never free new ops, but is confusing.  Explicitly push this into a
separate helper.

CID 739607 (#1-2 of 2): Read from pointer after free (USE_AFTER_FREE)
At (9): Dereferencing freed pointer "o".

CID 739606 (#1 of 1): Read from pointer after free (USE_AFTER_FREE)
At (28): Dereferencing freed pointer "op".

Signed-off-by: Sage Weil <sage@inktank.com>
src/osdc/Objecter.cc
src/osdc/Objecter.h