]> granicus.if.org Git - apache/commitdiff
merge of r1745175 from trunk:
authorStefan Eissing <icing@apache.org>
Mon, 23 May 2016 12:44:02 +0000 (12:44 +0000)
committerStefan Eissing <icing@apache.org>
Mon, 23 May 2016 12:44:02 +0000 (12:44 +0000)
mod_http2: fix for possible null pointer deref on aborted bucket beam.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1745176 13f79535-47bb-0310-9956-ffa450edef68

modules/http2/h2_bucket_beam.c
modules/http2/mod_h2.h [deleted file]

index d648b1d159db02b3ab93aec5b7b777cf7a2040a9..6b32e9beaba553734ac2f9714a8e10fe43486ae2 100644 (file)
@@ -738,7 +738,7 @@ apr_status_t h2_beam_receive(h2_bucket_beam *beam,
     if (enter_yellow(beam, &bl) == APR_SUCCESS) {
 transfer:
         if (beam->aborted) {
-            if (!APR_BRIGADE_EMPTY(beam->green)) {
+            if (beam->green && !APR_BRIGADE_EMPTY(beam->green)) {
                 apr_brigade_cleanup(beam->green);
             }
             status = APR_ECONNABORTED;
diff --git a/modules/http2/mod_h2.h b/modules/http2/mod_h2.h
deleted file mode 100644 (file)
index bb895dd..0000000
+++ /dev/null
@@ -1,19 +0,0 @@
-/* Copyright 2015 greenbytes GmbH (https://www.greenbytes.de)
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-#ifndef mod_h2_mod_h2_h
-#define mod_h2_mod_h2_h
-
-#endif