You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "Leif Hedstrom (JIRA)" <ji...@apache.org> on 2016/04/06 22:33:25 UTC
[jira] [Resolved] (TS-4231) Coverity issues in inlined plugin: CID
1352204 and CID 1352203
[ https://issues.apache.org/jira/browse/TS-4231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Leif Hedstrom resolved TS-4231.
-------------------------------
Resolution: Fixed
> Coverity issues in inlined plugin: CID 1352204 and CID 1352203
> --------------------------------------------------------------
>
> Key: TS-4231
> URL: https://issues.apache.org/jira/browse/TS-4231
> Project: Traffic Server
> Issue Type: Bug
> Components: Plugins
> Reporter: Leif Hedstrom
> Assignee: Daniel Vitor Morilha
> Fix For: 6.2.0
>
>
> {code}
> ** CID 1352204: Incorrect expression (USELESS_CALL)
> ________________________________________________________________________________________________________
> *** CID 1352204: Incorrect expression (USELESS_CALL)
> /plugins/experimental/inliner/ts.cc: 175 in ats::io::WriteOperation::Handle(tsapi_cont *, TSEvent, void *)()
> 169 case TS_EVENT_TIMEOUT:
> 170 TSError("[" PLUGIN_TAG "] TS_EVENT_TIMEOUT from producer");
> 171
> 172 handle_error:
> 173 operation.close();
> 174 assert(operation.action_ != NULL);
> CID 1352204: Incorrect expression (USELESS_CALL)
> Calling "TSActionDone(operation.action_)" is only useful for its return value, which is ignored.
> 175 TSActionDone(operation.action_);
> 176 operation.action_ = NULL;
> 177 /*
> 178 TSContDataSet(c, NULL);
> 179 delete p;
> 180 */
> ** CID 1352203: Error handling issues (CHECKED_RETURN)
> /plugins/experimental/inliner/ts.cc: 175 in ats::io::WriteOperation::Handle(tsapi_cont *, TSEvent, void *)()
> ________________________________________________________________________________________________________
> *** CID 1352203: Error handling issues (CHECKED_RETURN)
> /plugins/experimental/inliner/ts.cc: 175 in ats::io::WriteOperation::Handle(tsapi_cont *, TSEvent, void *)()
> 169 case TS_EVENT_TIMEOUT:
> 170 TSError("[" PLUGIN_TAG "] TS_EVENT_TIMEOUT from producer");
> 171
> 172 handle_error:
> 173 operation.close();
> 174 assert(operation.action_ != NULL);
> CID 1352203: Error handling issues (CHECKED_RETURN)
> Calling "TSActionDone" without checking return value (as is done elsewhere 8 out of 9 times).
> 175 TSActionDone(operation.action_);
> 176 operation.action_ = NULL;
> 177 /*
> 178 TSContDataSet(c, NULL);
> 179 delete p;
> 180 */
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)