Re: [BUG][MINOR] incorrect number of configuration files allowed

Hi Cyril,

On Thu, Dec 10, 2009 at 12:38:08AM +0100, Cyril Bonté wrote:
> Hi,
> I've noticed that haproxy (1.3/1.4 branches) accepts 1 more configuration file than allowed.
>
> The max default is fixed to 10 (#define MAX_CFG_FILES 10) but we can specify 11 "-f" parameters.
> This parameters are then stored in a 10 elements array.
>
> In haproxy.c, the test
> if (cfg_nbcfgfiles > MAX_CFG_FILES) {
> should be :
> if (cfg_nbcfgfiles >= MAX_CFG_FILES) {

Ah good catch. I'll change this. I'd prefer to take the time to implement an unlimited list, but it will be slightly longer to implement.

Thanks!
Willy Received on 2009/12/10 21:46