Subject: Re: [PATCH 0/5] First batch of fixes for Coverity errors

Re: [PATCH 0/5] First batch of fixes for Coverity errors

From: David Drysdale <drysdale_at_google.com>
Date: Mon, 22 Sep 2014 11:11:53 +0100

Thanks! The changes mostly look good to me; let's see if anyone else
has comments

On Fri, Sep 19, 2014 at 7:51 PM, Gregor Jasny <gjasny_at_googlemail.com> wrote:
> Hello,
>
> this is the first batch of patches for errors reported by Coverity.
>
> Please review and apply.
>
> Thanks,
> Gregor
>
> Gregor Jasny (5):
> Null check before dereference
> Fix integer shift overflow if both tcp_socket and udp_socket are set
> Free temporary variable in error path
> Re-start loop if select fails
> Prevent tmpbuf from overrunning
>
> acountry.c | 7 ++++++-
> ares_getnameinfo.c | 30 ++++++++++++++++++++----------
> ares_getsock.c | 8 +++-----
> ares_options.c | 3 +++
> 4 files changed, 32 insertions(+), 16 deletions(-)
>
> --
> 1.8.5.2 (Apple Git-48)
>
Received on 2014-09-22