New: backend_connect() bind behavior wrong

h
ttp://bugzilla.andrew.cmu.edu/show_bug.cgi?id=2864

           Summary: backend_connect() bind behavior wrong
           Product: Cyrus IMAP
           Version: 2.3.x
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: bug
          Priority: P2
         Component: Murder
        AssignedTo: murchandrew.cmu.edu
        ReportedBy: wesumich.edu
         QAContact: cyrus-bugzillalists.andrew.cmu.edu


There are two problems with the call to bind() in
backend_connect().  First, if res0 (the target host) has 
more than one address, then the second time through the res0
loop, res1 will have been freed().  Second, 
if res1 has multiple addresses, the bind( res1 ) may choose
an inappropriate address.  To alleviate these 
problems, I suggest removing the bind(), the free(), and the
getaddrinfo() on res1.  Since this call is 
building a connection from the current host to a backend,
the bind() should be extraneous.  What was the 
intent of including it in the first place?

:w



------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA
contact.