OSEC

Neohapsis is currently accepting applications for employment. For more information, please visit our website www.neohapsis.com or email hr@neohapsis.com
Re: bounce_size_limit was broken

From: YOSHIMURA Keitaro (ramsyramix.jp)
Date: Fri May 16 2008 - 10:50:08 CDT


Hi.

> In what way is it "amusing"? Please describe your use case.
If set to 1, original mail(header & body) was full attached.

> Zero means "no limit". The patch breaks this (though in practice main.cf
> settings are restricted to >= 1, so this breaks a use case that is
> currently disabled).
No, 0 is a value not correct.

src/bounce/bounce.c:
    static CONFIG_INT_TABLE int_table[] = {
      VAR_BOUNCE_LIMIT, DEF_BOUNCE_LIMIT, &var_bounce_limit, 1, 0,
      0,
    };

May 16 11:48:04 XXXXXX postfix/bounce[28141]: fatal: invalid bounce_size_limit parameter value 0 < 1

--
<|> YOSHIMURA Keitaro/ramsy
<|> ramsyramix.jp
<|> http://ramix.jp/~ramsy/