cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: bagder: curl/ares ares_process.c,1.10,1.11

From: Daniel Stenberg <daniel-curl_at_haxx.se>
Date: Sat, 31 Jul 2004 09:12:39 +0200 (CEST)

On Fri, 30 Jul 2004, Gisle Vanem wrote:

>> @@ -143,16 +144,16 @@
>> /* Can't allocate iovecs; just send the first request. */
>> sendreq = server->qhead;
>>
>> - count = send(server->tcp_socket, sendreq->data, sendreq->len, 0);
>> + scount = send(server->tcp_socket, sendreq->data, sendreq->len, 0);
>
> I fail to see the point of the else-part. If allocating N iovecs fails,
> we're toast anyway. send() may allocate process memory too. And later on
> anyway when the answers should be processed.

Reading this code again, I can only agree with you. Let's cut out the else
part and have it fail normally if malloc() fails.

> Is there a story behind this?

Probably, but since the original author isn't reading this I bet we won't get
any info on that... :-(

-- 
      Daniel Stenberg -- http://curl.haxx.se -- http://daniel.haxx.se
       Dedicated custom curl help for hire: http://haxx.se/curl.html
Received on 2004-07-31