elecharny
commented
2 years ago Hi Artur,
it's more complicated that what I thought. Actually, the handling of the OPTS_MLST command is not properly done. Looking at the RFC, there are syntax errors that are not checked. Looking at, we should get an error when sending such a request:
C> opts MLst unique type; S> 501 Invalid MLST options
Actually, we don't...
Hi Arturo,
for the OPTS_MLST patch, I think the logic would be to check if the returned array is not empty, and if so returning an error. The current code check if it's NULL but I think it's a mistake. So your suggestion is correct (logically) but I'd rather change the 'if' and keep it.