[trafficserver] branch master updated: Coverity 1373300: Use of untrusted scalar value

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

masaori 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  f45e257   Coverity 1373300: Use of untrusted scalar value
f45e257 is described below

commit f45e2574d6a298a962e1f032f5855179ee1295c3
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Tue May 30 11:38:14 2017 +0900

    Coverity 1373300: Use of untrusted scalar value
---
 proxy/http2/Http2ConnectionState.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/proxy/http2/Http2ConnectionState.h b/proxy/http2/Http2ConnectionState.h
index 8942065..3ba9af4 100644
--- a/proxy/http2/Http2ConnectionState.h
+++ b/proxy/http2/Http2ConnectionState.h
@@ -81,7 +81,7 @@ public:
   unsigned
   set(Http2SettingsIdentifier id, unsigned value)
   {
-    if (id < HTTP2_SETTINGS_MAX) {
+    if (0 < id && id < HTTP2_SETTINGS_MAX) {
       return this->settings[indexof(id)] = value;
     } else {
       ink_assert(!"Bad Settings Identifier");

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].