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

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

From: David Drysdale <drysdale_at_google.com>
Date: Tue, 30 Sep 2014 17:21:57 +0100

As no-one else objected, I've pushed these to GitHub. Thanks for the fixes!

On Sun, Sep 28, 2014 at 9:19 PM, Gregor Jasny <gjasny_at_googlemail.com> wrote:

> this is the first batch of patches for errors reported by Coverity.
>
> I addressed the review comments from David Drysdale.
>
> 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 | 20 ++++++++++++++------
> ares_getsock.c | 8 +++-----
> ares_options.c | 3 +++
> 4 files changed, 26 insertions(+), 12 deletions(-)
>
> --
> 1.9.3 (Apple Git-50)
>
>
Received on 2014-09-30