Details
-
Bug
-
Resolution: Fixed
-
Critical
-
3.0
-
Security Level: Public
-
None
-
Untriaged
-
Unknown
Description
Bug reported by the clang static analyzer.
Description: Function call argument is an uninitialized value
File: /Users/dave/repos/couchbase/server/source/memcached/daemon/memcached.c upr_stream_req_executor()
Line: 4242
See attached report.
From speaking to Trond offline he believes that it shouldn't be possible to enter upr_stream_req_executor() with c->aiostat == ENGINE_ROLLBACK (which is what triggers this error) - in which case we should just add a suitable assert() to squash the warning.
Attachments
For Gerrit Dashboard: MB-11326 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
38560,4 | MB-11326: Not possible to enter dcp_stream_req_executor() with ENGINE_ROLLBACK | 3.0 | memcached | Status: MERGED | +2 | +1 |
39779,1 | MB-11326: Not possible to enter dcp_stream_req_executor() with ENGINE_ROLLBACK | master | memcached | Status: ABANDONED | 0 | 0 |
39809,1 | Merge remote-tracking branch 'membase/3.0' | 3.0.1 | memcached | Status: MERGED | +2 | +1 |
39810,1 | Merge remote-tracking branch 'membase/3.0.1' | master | memcached | Status: MERGED | +2 | +1 |