← Back to team overview

maria-developers team mailing list archive

Re: Please review MDEV-9604 crash in Item::save_in_field with empty enum value

 

Hi, Alexander!

On Mar 22, Alexander Barkov wrote:
> Hi Sergei,
> 
> It fixes the reported crash, and additionally
> inconsistencies when comparing temporal columns to empty strings:
> 
> In some cases it returned rows, in other cases it returned no rows,
> depending on:
> - the engine
> - the type of comparison arguments (columns vs literals).
> - presence of an index
> 
> Now everything works in the same way.
> 
> Note, this additional problem is not 10.1 specific, it existed in 10.0.
> But 10.0 and 10.1 worked differently in certain cases.

Ok to push. In 10.0, right?

And, by the way, please send a full commit next time, not just a diff.
It was quite a challenge to figure out what was the bug and what were
you fixing without a commit comment :(

Regards,
Sergei
Chief Architect MariaDB
and security@xxxxxxxxxxx


Follow ups

References