Re: ws2_32: Avoid writing to NULL pointer (coverity)

[Alexandre Julliard]

Bruno Jesus <00cpxxx(a)gmail.com> writes:
> Due to the format set in sprintf this is impossible to happen, but
> coverity is not content aware and complains that the strstr may return
> NULL. This patch should make it happy.

If it can't happen there's no reason to add a check.

-- 
Alexandre Julliard
julliard(a)winehq.org

ws2_32: Avoid writing to NULL pointer (coverity)

[Bruno Jesus]

On Thu, Dec 5, 2013 at 8:00 AM, Alexandre Julliard <julliard(a)winehq.org> wrote:
> Bruno Jesus <00cpxxx(a)gmail.com> writes:
> > Due to the format set in sprintf this is impossible to happen, but
> > coverity is not content aware and complains that the strstr may return
> > NULL. This patch should make it happy.
>
> If it can't happen there's no reason to add a check.

I was aware that this could be your answer, I just double checked by
sending the patch to mark the correct resolution in the coverity site.
> --
> Alexandre Julliard
> julliard(a)winehq.org

Bruno