github_mirrors/ModSecurity
3
0
Fork 1
mirror of https://github.com/owasp-modsecurity/ModSecurity.git synced 2025-09-29 19:24:29 +03:00
Code Issues Packages Projects Releases Wiki Activity

Do not add alert message when a transformation is not specified in a Lua rule.

Browse Source
This commit is contained in:
brectanus
2008-02-11 21:57:21 +00:00
parent 2c74c55d4d
commit 935677a780
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
apache2/msc_lua.c
View File

@@ -140,7 +140,10 @@ static apr_array_header_t *resolve_tfns(lua_State *L, int idx, modsec_rec *msr,
tfn_arr = apr_array_make(mp, 25, sizeof(msre_tfn_metadata *)); tfn_arr = apr_array_make(mp, 25, sizeof(msre_tfn_metadata *));
if (tfn_arr == NULL) return NULL; if (tfn_arr == NULL) return NULL;
if (lua_istable(L, idx)) { /* Is the second parameter an array? */ /* ENH: Why is this userdata and not none/nil when parameter not given? */
if (lua_isuserdata(L, idx) || lua_isnoneornil(L, idx)) { /* No second parameter */
return tfn_arr;
} else if (lua_istable(L, idx)) { /* Is the second parameter an array? */
int i, n = lua_objlen(L, idx); int i, n = lua_objlen(L, idx);
for(i = 1; i <= n; i++) { for(i = 1; i <= n; i++) {
@@ -176,7 +179,7 @@ static apr_array_header_t *resolve_tfns(lua_State *L, int idx, modsec_rec *msr,
} }
} }
} else { } else {
msr_log(msr, 1, "SecRuleScript: Transformation parameter must be a transformation name or array of transformation names."); msr_log(msr, 1, "SecRuleScript: Transformation parameter must be a transformation name or array of transformation names, but found \"%s\" (type %d).", lua_typename(L, idx), lua_type(L, idx));
return NULL; return NULL;
} }