[Bug 909] New: connect() uses "wrong" addrlen in dcc.c

View: New views
2 Messages — Rating Filter:   Alert me  

[Bug 909] New: connect() uses "wrong" addrlen in dcc.c

by Bugzilla from wbreyha@gmx.net :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=909
           Summary: connect() uses "wrong" addrlen in dcc.c
           Product: Exim
           Version: 4.69
          Platform: Other
        OS/Version: Linux
            Status: NEW
          Severity: bug
          Priority: medium
         Component: ACLs
        AssignedTo: nigel@...
        ReportedBy: wbreyha@...
                CC: exim-dev@...


Created an attachment (id=343)
 --> (http://bugs.exim.org/attachment.cgi?id=343)
use sizeof(serv_addr) for connect()

A user reported troubles with my dcc patch on 64bit FreeBSD.

I used
servlen = Ustrlen(serv_addr.sun_path) + sizeof(serv_addr.sun_family);
connect(sockfd, (struct sockaddr *) &serv_addr, servlen);

That doesn't work on all plattforms. I looked at some other parts of exim and
changed that to
connect(sockfd, (struct sockaddr *) &serv_addr, sizeof(serv_addr));
accordingly.

Greetings, Wolfgang


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email

--
## List details at http://lists.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##

[Bug 909] connect() uses "wrong" addrlen in dcc.c

by Bugzilla from nigel@exim.org :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=909

Nigel Metheringham <nigel@...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #1 from Nigel Metheringham <nigel@...>  2009-11-12 08:34:24 ---
CVS commit by nm4:

Correct connect() call in dcc code.  Fixes: bug#909

--- CVS commit summary ---
1.589  1.590  +2 -0  -  exim/exim-doc/doc-txt/ChangeLog
1.4    1.5    +2 -3  -  exim/exim-src/src/dcc.c


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email

--
## List details at http://lists.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##