effect_list_sanity_cb() uses log_error() instead of ruleset_error()

• 報告者: cazfi
• 担当者: cazfi
• チケットの種類: バグ
• 状況: 完了
• コンポーネント: Server
• マイルストーン: 3.0.4 (完了済み)
• 優先度: 5 - 中
• 重要度: 5 - 中
• 解決法: 修正済み
• ファイル: 3

詳細

effect_list_sanity_cb() reports some errors with log_error() instead of ruleset_error(). That's likely a bug.

チケットの履歴 (9 件中 3 件表示)

2022-04-14 16:03 更新者: cazfi

• 新しいチケット "effect_list_sanity_cb() uses log_error() instead of ruleset_error()" が作成されました

2022-05-29 07:26 更新者: cazfi

• マイルストーン が (未割り当て) から 3.1.0 (完了済み) に更新されました

コメント

rssanity.c does the same on number of places outside effect_list_sanity_cb()

As the ruleset_error() currently stands, there's not much difference between ruleset_error(LOG_ERROR ...) and log_error() -> this is probably not worth fixing in S3_0.

The main difference between those functions is that ruleset_error() has a buffer of just 1024 bytes, and do_log() that log_error() uses buffer size is MAX_LEN_LOG_LINE (5120), so changing those calls to ruleset_error() would actually risk truncating them. Will open a new ticket about increasing ruleset_error() buffer size.

2022-05-29 07:38 更新者: cazfi

コメント

Reply To cazfi

Will open a new ticket about increasing ruleset_error() buffer size.

-> #44704, consider that one dependency of this.

2022-08-22 10:04 更新者: cazfi

コメント

Reply To cazfi

As the ruleset_error() currently stands, there's not much difference between ruleset_error(LOG_ERROR ...) and log_error() -> this is probably not worth fixing in S3_0.

This might change with #45395, in a way that makes this ticket its dependency (should properly call ruleset_error() before assuming change in ruleset_error() behavior to take effect)

2022-08-22 11:43 更新者: cazfi

• 添付ファイル 0049-rssanity-Use-ruleset_error-instead-of-log_error.patch (File ID: 10130) が付加されました

2022-08-22 11:44 更新者: cazfi

• 添付ファイル 0034-rssanity-Use-ruleset_error-instead-of-log_error.patch (File ID: 10131) が付加されました

2022-08-22 11:44 更新者: cazfi

• 添付ファイル 0019-rssanity-Use-ruleset_error-instead-of-log_error.patch (File ID: 10132) が付加されました

2022-08-22 11:44 更新者: cazfi

• 担当者 が (未割り当て) から cazfi に更新されました
• 解決法 が なし から 受領 に更新されました
• マイルストーン が 3.1.0 (完了済み) から 3.0.4 (完了済み) に更新されました

2022-08-30 14:01 更新者: cazfi

• 状況 が オープン から 完了 に更新されました
• 解決法 が 受領 から 修正済み に更新されました