X-Recipient:	archive-cygwin AT delorie DOT com
DomainKey-Signature:	a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:message-id:date:from:mime-version:to:subject
	:references:in-reply-to:content-type:content-transfer-encoding;
	q=dns; s=default; b=HRVz0YNxSilpRMVX7In3XRA/2FTtEVFZeiSU7Vnh0P1
	FAla6cvDzyd28Nbk6yoDnXi8DRL34WqEhPEd8ZDTb1ucmk98a5lgI9P/ft0RcMCF
	wRptnvGQTOJWiiI63vhvhS3y7flX9Hi2pJLJg49mqsCqh1bAa8ZR2fie+Rno0+2Y
	=
DKIM-Signature:	v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:message-id:date:from:mime-version:to:subject
	:references:in-reply-to:content-type:content-transfer-encoding;
	s=default; bh=MiRoEsKy8bIHkakhDRnNFyF5FIE=; b=QpI4DBBvmNVIxbo8E
	5kQqTu1U57NY2OgOBgALt4rXzZ3cbm9AU9Qu7tb+cU6ZCoKmdZgb91JyK6XJUBS6
	sZ1SZnFxDru0060qVum/Nr6JGVb+LACv71XGohucMX4rcKCk9ncP4zpCmIPoAw0j
	cJsvZcFw7ega5hSBJZiQPDYhoY=
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
Authentication-Results:	sourceware.org; auth=none
X-Virus-Found:	No
X-Spam-SWARE-Status:	No, score=-1.6 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2
X-HELO:	mail-wg0-f41.google.com
X-Received:	by 10.180.77.165 with SMTP id t5mr10760347wiw.38.1398589527664; Sun, 27 Apr 2014 02:05:27 -0700 (PDT)
Message-ID:	<535CC856.7050808@gmail.com>
Date:	Sun, 27 Apr 2014 11:05:26 +0200
From:	jdzstz <jdzstz AT gmail DOT com>
User-Agent:	Mozilla/5.0 (Windows NT 6.3; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0
MIME-Version:	1.0
To:	cygwin AT cygwin DOT com
Subject:	Bug at getsockopt when TCP_NODELAY is used as parameter
References:	<CA+wAnvyrB2ecwdJsvzqmMNjPFTnW6rT0rYdNcXRbK4z5WJqwyg AT mail DOT gmail DOT com>
In-Reply-To:	<CA+wAnvyrB2ecwdJsvzqmMNjPFTnW6rT0rYdNcXRbK4z5WJqwyg@mail.gmail.com>
X-Forwarded-Message-Id:	<CA+wAnvyrB2ecwdJsvzqmMNjPFTnW6rT0rYdNcXRbK4z5WJqwyg AT mail DOT gmail DOT com>
X-IsSubscribed:	yes

While compiling varnish 3.0.5, I have detected a possible bug at 
getsockopt when TCP_NODELAY is used as parameter. The issue is caused 
because at cygwin it returns a BOOL, instead returning INT value like at 
Linux.

At varnish 3.0.5, the following code at "sock_test(int fd)" uses getsockopt:
    - 
https://www.varnish-cache.org/trac/browser/bin/varnishd/cache_acceptor.c?rev=dec9098a2ac760d7ba4765a9a1a1091103b6eb05

     l = sizeof tcp_nodelay;
     i = getsockopt(fd, IPPROTO_TCP, TCP_NODELAY, &tcp_nodelay, &l);
     if (i) {
         VTCP_Assert(i);
         return;
     }
     assert(l == sizeof tcp_nodelay);
     if (!tcp_nodelay)
         need_tcpnodelay = 1;


The problem is that at cygwin, this ASSERT fails:
     -     assert(l == sizeof tcp_nodelay);

Because before function call, the initial tcp_nodelay variable size is 
INT, but after funtion call, it returns a pointer to a BOOL variable, so 
assert (sizeof INT == sizeof BOOL) fails.

After some investigations I have seen that at windows, its getsockopt 
function, returns BOOL for TCP_NODELAY:
     - 
http://msdn.microsoft.com/en-us/library/windows/desktop/ms738544%28v=vs.85%29.aspx

But at Unix, the correct behaviour is to return INT:
     - http://www.yolinux.com/TUTORIALS/Sockets.html

I suppose that the cygwin getsockopt internally calls to Windows 
getsockopt, and no type conversion of variables is done. I think the 
solution would be to wrap getsockopt response and change it from bool 
size variable to a int size variable.

As a temporary workaround for varnish 3.0.5 I have changed the variable 
types to bool, but I think It would be better to directly fix it at 
cygwin layer, to avoid modifications to original Linux code:

--- origsrc/varnish-3.0.5/bin/varnishd/cache_acceptor.c 2013-12-02 
08:48:15.000000000 +0100
+++ src/varnish-3.0.5/bin/varnishd/cache_acceptor.c 2014-04-17 
21:31:12.555512600 +0200
@@ -36,6 +36,9 @@
  #include <string.h>
  #include <stdlib.h>
  #include <unistd.h>
+#if defined(__CYGWIN__)
+#include <stdbool.h>
+#endif

  #include <sys/uio.h>
  #include <sys/types.h>
@@ -105,7 +108,12 @@ sock_test(int fd)
      struct linger lin;
      struct timeval tv;
      socklen_t l;
-    int i, tcp_nodelay;
+    int i;
+    #if defined(__CYGWIN__)
+    bool tcp_nodelay;
+    #else
+    int tcp_nodelay;
+    #endif

      l = sizeof lin;
      i = getsockopt(fd, SOL_SOCKET, SO_LINGER, &lin, &l);
@@ -172,7 +180,11 @@ VCA_Prep(struct sess *sp)
  {
      char addr[VTCP_ADDRBUFSIZE];
      char port[VTCP_PORTBUFSIZE];
+    #if defined(__CYGWIN__)
+    bool tcp_nodelay = true;
+    #else
      int tcp_nodelay = 1;
+    #endif

      VTCP_name(sp->sockaddr, sp->sockaddrlen,
          addr, sizeof addr, port, sizeof port);


Thank you
Jorge








---
This email is free from viruses and malware because avast! Antivirus protection is active.
http://www.avast.com


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

- Raw text -