From 6c5d19ed84999430de883478557c116349f4bbfa Mon Sep 17 00:00:00 2001 From: ivanr Date: Thu, 22 Feb 2007 11:40:48 +0000 Subject: [PATCH] Removed the (harmless) message mentioning LAST_UPDATE_TIME missing. --- CHANGES | 2 ++ apache2/re_actions.c | 4 +++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/CHANGES b/CHANGES index 7130894e..0ac5f02a 100644 --- a/CHANGES +++ b/CHANGES @@ -2,6 +2,8 @@ 22 Feb 2006 - 2.1.0-rc7+ ------------------------ +* Removed the (harmless) message mentioning LAST_UPDATE_TIME missing. + * It was not possible to remove a rule placed in phase 4 using SecRuleRemoveById or SecRuleRemoveByMsg. Fixed. diff --git a/apache2/re_actions.c b/apache2/re_actions.c index f6c8b230..460febc2 100644 --- a/apache2/re_actions.c +++ b/apache2/re_actions.c @@ -1100,7 +1100,9 @@ static apr_status_t msre_action_deprecatevar_execute(modsec_rec *msr, apr_pool_t /* Find the last update time (of the collection). */ var_last_update_time = (msc_string *)apr_table_get(target_col, "LAST_UPDATE_TIME"); if (var_last_update_time == NULL) { - msr_log(msr, 1, "Internal Error: Collection missing LAST_UPDATE_TIME."); + /* This is all right. If collection was created (and not restored from + * storage) then it won't have LAST_UPDATE_TIME - it was never updated. + */ return 0; }