This is an automated email from the ASF dual-hosted git repository.
zwoop pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push:
new e22113d Coverity 1022104: Dereference before null check on _s
e22113d is described below
commit e22113d32231d3586e639a2973b98b6845f9473d
Author: Leif Hedstrom <[email protected]>
AuthorDate: Sat May 27 16:07:43 2017 -0600
Coverity 1022104: Dereference before null check on _s
---
proxy/http/remap/RemapPlugins.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/proxy/http/remap/RemapPlugins.cc b/proxy/http/remap/RemapPlugins.cc
index 410fb88..c715ad9 100644
--- a/proxy/http/remap/RemapPlugins.cc
+++ b/proxy/http/remap/RemapPlugins.cc
@@ -55,7 +55,7 @@ RemapPlugins::run_plugin(remap_plugin_info *plugin)
_s->remap_plugin_instance = ih;
}
- plugin_retcode = plugin->fp_tsremap_do_remap(ih, _s ?
reinterpret_cast<TSHttpTxn>(_s->state_machine) : nullptr, &rri);
+ plugin_retcode = plugin->fp_tsremap_do_remap(ih,
reinterpret_cast<TSHttpTxn>(_s->state_machine), &rri);
// TODO: Deal with negative return codes here
if (plugin_retcode < 0) {
plugin_retcode = TSREMAP_NO_REMAP;
--
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].
