[
https://issues.apache.org/jira/browse/TS-4572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15344648#comment-15344648
]
ASF GitHub Bot commented on TS-4572:
------------------------------------
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/728
Fwiw, this is where I completely disagree with jpeach, an assert with side
effects is simply wrong. Even if it's a release assert. I'd be ok if we changed
the name of the TSReleaseAssert() to something that's not "assert". The real
danger is that someone relaxes it from a release assert to a debug assert, and
then it'll obviously fail completely.
> Fix "Unchecked return value" (CID 1356972)
> ------------------------------------------
>
> Key: TS-4572
> URL: https://issues.apache.org/jira/browse/TS-4572
> Project: Traffic Server
> Issue Type: Bug
> Components: Plugins
> Reporter: Phil Sorber
> Assignee: Phil Sorber
> Fix For: 7.0.0
>
>
> {noformat}
> If the function returns an error value, the error value may be mistaken for a
> normal value.
> In get_effective_host: Value returned from a function is not checked for
> errors before being used (CWE-252)
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
