From: Aaron Bannert <aaron@php.net>
Date: Thu, 18 Apr 2002 16:25:50 +0000 (+0000)
Subject: It makes more sense to do the null-pointer check *before* trying to use it.
X-Git-Tag: php-4.2.0~9
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=23390048f025001fc66dff3c98d7bbc5dfca3bb3;p=php

It makes more sense to do the null-pointer check *before* trying to use it.
(Also fix a typo that Cliff pointed out: "safe" --> "save".)

Obtained from:  Ryan Morgan <rmorgan@covalent.net>
---

diff --git a/sapi/apache2filter/sapi_apache2.c b/sapi/apache2filter/sapi_apache2.c
index 24724bb9b7..856d146c34 100644
--- a/sapi/apache2filter/sapi_apache2.c
+++ b/sapi/apache2filter/sapi_apache2.c
@@ -346,12 +346,12 @@ static int php_output_filter(ap_filter_t *f, apr_bucket_brigade *bb)
 	ap_add_cgi_vars(f->r);
 
 	ctx = SG(server_context);
-	ctx->f = f; /* safe whatever filters are after us in the chain. */
 	if (ctx == NULL) {
 		ap_log_rerror(APLOG_MARK, APLOG_ERR|APLOG_NOERRNO, 0, f->r,
 					 "php failed to get server context");
         return HTTP_INTERNAL_SERVER_ERROR;
 	}
+	ctx->f = f; /* save whatever filters are after us in the chain. */
 
 	if (ctx->request_processed) {
 		return ap_pass_brigade(f->next, bb);