Erik Sjoelund wrote:
> 
> It looks strange in qmail-local.c as of qmail 1.03
> 
> line 648 in qmail-local.c  is written
>      while ((k > i) && (cmds.s[k - 1] == ' ') || (cmds.s[k - 1] ==
> '\t'))
>        cmds.s[--k] = 0;
> 
> Because && has higher precedence than ||,
> I suppose that was meant to be written was
> 
>     while ((k > i) && ((cmds.s[k - 1] == ' ') || (cmds.s[k - 1] ==
> '\t')))
>        cmds.s[--k] = 0;
> 
> this could lead to strange behaviour for a  .qmail starting
> with just tabs followed by a newline. As  strange behaviour I
> mean referencing the array with index -1
> 
> Sorry if I'm mistaken, please correct me if I'm wrong
> 
> thanks & regards
> Erik Sjoelund

i just tested it with a .qmail file only containing one line consisting of 3 tabs and 
a newline...

the log says:

Oct 11 19:53:02 mother qmail: 971286782.258799 delivery 3: deferral:
Uh-oh:_first_line_of_.qmail_file_is_blank._(#4.2.1)/

/Martin

Reply via email to