On 28/11/2019 14:00, Andreas Beckmann wrote:
> On 28/11/2019 13.39, David Bürgin wrote:
>> Given all this, I want to propose changing the log level of this message
>> to debug (SMI_LOG_DEBUG), guarded by ‘if (dbg > 0)’ as elsewhere.
> 
> Your reasoning sounds plausible. Can you provide a patch?

Sure, it’s a tiny patch. Thank you!

---
 libmilter/listener.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libmilter/listener.c b/libmilter/listener.c
index 11d92bb..d009621 100644
--- a/libmilter/listener.c
+++ b/libmilter/listener.c
@@ -979,8 +979,8 @@ mi_listener(conn, dbg, smfi, timeout, backlog)
                mi_stop_milters(MILTER_ABRT);
        else
        {
-               if (mistop != MILTER_CONT)
-                       smi_log(SMI_LOG_INFO, "%s: mi_stop=%d",
+               if (mistop != MILTER_CONT && dbg > 0)
+                       smi_log(SMI_LOG_DEBUG, "%s: mi_stop=%d",
                                smfi->xxfi_name, mistop);
                mi_closener();
        }
-- 
2.17.1

Reply via email to