From: Daniel Gruno <humbedooh@apache.org>
Date: Fri, 28 Mar 2014 20:33:08 +0000 (+0000)
Subject: mod_lua: Since c->notes is the only place where r will be NULL, we don't need to... 
X-Git-Tag: 2.5.0-alpha~4378
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=29e243b1fb3e870b9cbb332b82e7224a298ca914;p=apache

mod_lua: Since c->notes is the only place where r will be NULL, we don't need to check a second time, since we'll never meddle with tables named 'notes'

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1582877 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/modules/lua/lua_apr.c b/modules/lua/lua_apr.c
index 70158fad79..21651ead4f 100644
--- a/modules/lua/lua_apr.c
+++ b/modules/lua/lua_apr.c
@@ -48,11 +48,9 @@ static int lua_table_set(lua_State *L)
         while ( (badchar = ap_strchr(badchar, '\n')) ) {
             *badchar = ' ';
         }
-        if (t->r != NULL) {
-            ap_log_rerror(APLOG_MARK, APLOG_WARNING, 0, t->r, 
+        ap_log_rerror(APLOG_MARK, APLOG_WARNING, 0, t->r, 
                 APLOGNO(02614) "mod_lua: Value for '%s' in table '%s' contains newline!",
                   key, t->n);
-        }
         apr_table_set(t->t, key, replacement);
     }
     else {