patch: fix sql plugin crash

Maxim Gorbachyov maxim.gorbachyov at gmail.com
Wed Oct 29 04:59:58 EDT 2008


On Tue, Oct 28, 2008 at 10:32 PM, Alexey Melnikov
<alexey.melnikov at isode.com> wrote:

> This looks reasonable.
>
> However, I am wondering if the following line:
>
> +    memset(value, 0, sizeof(value));
>
> can be replaced with:
>
> +    value[0] = '\0';
>
> Can you test?

Tested. It works.

> Also, please add this to bugzilla, if you haven't yet.

That's done, but with initial version of the patch attached:
http://bugzilla.andrew.cmu.edu/show_bug.cgi?id=3111


More information about the Cyrus-devel mailing list