[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] Various fixes for resolve_host, from src/c/api/libspeechd.c.
From: |
Boris Dusek |
Subject: |
[PATCH] Various fixes for resolve_host, from src/c/api/libspeechd.c. |
Date: |
Wed, 8 Sep 2010 22:06:21 +0200 |
Hello Chris,
On Sep 7, 2010, at 4:23 PM, Christopher Brannon wrote:
> * Check return values of malloc and inet_ntop. Do not proceed if they
> are NULL.
> * Store the return value of inet_ntop in a const char * variable, since
> the function returns a pointer to constant data.
> ---
> src/c/api/libspeechd.c | 20 ++++++++++++++++----
> 1 files changed, 16 insertions(+), 4 deletions(-)
I can confirm that this compiles without a warning on amd64.
Cannot comment on the correctness of the code itself though.
Boris