You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2016/05/31 13:46:01 UTC
[trafficserver] branch 6.2.x updated (262783d -> 65734ed)
This is an automated email from the ASF dual-hosted git repository.
sorber pushed a change to branch 6.2.x
in repository https://git-dual.apache.org/repos/asf/trafficserver.git.
from 262783d TS-3123 Make proxy.config.http.transaction_active_timeout_in overridable
new bf91c3c TS-3123: Missing NULL check and failing clang-analyzer
new 65734ed TS-3123: Missing break statement and failing clang-analyzer
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.
Summary of changes:
proxy/InkAPI.cc | 1 +
proxy/http/HttpTransact.cc | 4 +++-
2 files changed, 4 insertions(+), 1 deletion(-)
--
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].
[trafficserver] 01/02: TS-3123: Missing NULL check and failing
clang-analyzer
Posted by so...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
sorber pushed a commit to branch 6.2.x
in repository https://git-dual.apache.org/repos/asf/trafficserver.git
commit bf91c3c7eec07eb651458f35a5e326a667ebe8ee
Author: Bryan Call <bc...@apache.org>
AuthorDate: Mon May 23 17:11:04 2016 -0700
TS-3123: Missing NULL check and failing clang-analyzer
(cherry picked from commit 006d63653f9bfd65a041adee7d4c7b2d43737428)
---
proxy/http/HttpTransact.cc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index 87e1771..aa4812e 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -917,7 +917,9 @@ HttpTransact::EndRemapRequest(State *s)
done:
// We now set the active-timeout again, since it might have been changed as part of the remap rules.
- s->state_machine->ua_session->get_netvc()->set_active_timeout(HRTIME_SECONDS(s->txn_conf->transaction_active_timeout_in));
+ if (s->state_machine->ua_session) {
+ s->state_machine->ua_session->get_netvc()->set_active_timeout(HRTIME_SECONDS(s->txn_conf->transaction_active_timeout_in));
+ }
if (is_debug_tag_set("http_chdr_describe") || is_debug_tag_set("http_trans") || is_debug_tag_set("url_rewrite")) {
DebugTxn("http_trans", "After Remapping:");
--
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.
[trafficserver] 02/02: TS-3123: Missing break statement and failing
clang-analyzer
Posted by so...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
sorber pushed a commit to branch 6.2.x
in repository https://git-dual.apache.org/repos/asf/trafficserver.git
commit 65734ed4e69bd259e2c8273823951064b911bc98
Author: Bryan Call <bc...@apache.org>
AuthorDate: Mon May 23 16:29:08 2016 -0700
TS-3123: Missing break statement and failing clang-analyzer
(cherry picked from commit f38eb6419574fdad4f1bd3e3bb082d2da296d5a9)
---
proxy/InkAPI.cc | 1 +
1 file changed, 1 insertion(+)
diff --git a/proxy/InkAPI.cc b/proxy/InkAPI.cc
index 55784bb..2a464d3 100644
--- a/proxy/InkAPI.cc
+++ b/proxy/InkAPI.cc
@@ -7974,6 +7974,7 @@ _conf_to_memberp(TSOverridableConfigKey conf, OverridableHttpConfigParams *overr
case TS_CONFIG_HTTP_PARENT_PROXY_TOTAL_CONNECT_ATTEMPTS:
typ = OVERRIDABLE_TYPE_INT;
ret = &overridableHttpConfig->parent_connect_attempts;
+ break;
case TS_CONFIG_HTTP_TRANSACTION_ACTIVE_TIMEOUT_IN:
typ = OVERRIDABLE_TYPE_INT;
ret = &overridableHttpConfig->transaction_active_timeout_in;
--
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.