You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2016/03/02 05:59:18 UTC
[jira] [Commented] (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:comment-tabpanel&focusedCommentId=15175031#comment-15175031 ]
ASF subversion and git services commented on TS-4231:
-----------------------------------------------------
Commit ed39fd447fda1c9a6a38d7ab84ceafe346ba6948 in trafficserver's branch refs/heads/master from [~dmorilha]
[ https://git-wip-us.apache.org/repos/asf?p=trafficserver.git;h=ed39fd4 ]
TS-4231: get rid of useless TSActionDone call
This closes #506.
> 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)