Bug 129260 - [patch] mail/exim - reinitialize len before calling accept()
Summary: [patch] mail/exim - reinitialize len before calling accept()
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Kirill Ponomarev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-11-28 16:50 UTC by Maxim Dounin
Modified: 2008-12-12 17:40 UTC (History)
0 users

See Also:


Attachments
patch-exim-accept.txt (623 bytes, patch)
2008-11-28 16:50 UTC, Maxim Dounin
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Maxim Dounin 2008-11-28 16:50:01 UTC
In exim accept loop reinitialization of len argument of accept() missing
between calls when working with multiple listening sockets (as of exim 4.69).
As a result some connections may be treated by exim as connections from
unrelated ip addresses.

More information in exim's bugzilla:

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

Fix: The following patch fixes the problem.  I've submitted this fix to upstream,
but probably adding this patch to FreeBSD ports tree is a good idea.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2008-11-28 16:50:16 UTC
Responsible Changed
From-To: freebsd-ports-bugs->krion

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Maxim Dounin 2008-12-12 17:25:24 UTC
Hello!

Just a quick note that the above patch was committed into exim source 
tree.

Maxim Dounin
Comment 3 dfilter service freebsd_committer freebsd_triage 2008-12-12 17:34:37 UTC
skv         2008-12-12 17:34:28 UTC

  FreeBSD ports repository

  Modified files:
    mail/exim            Makefile 
  Added files:
    mail/exim/files      patch-src-daemon.c 
  Log:
  Bugfix: reinitialize len before calling accept().
  
  PR:             ports/129260
  Submitted by:   Maxim Dounin <mdounin xx mdounin.ru>
  Bug:            http://bugs.exim.org/show_bug.cgi?id=770
  
  Revision  Changes    Path
  1.240     +1 -1      ports/mail/exim/Makefile
  1.1       +19 -0     ports/mail/exim/files/patch-src-daemon.c (new)
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 4 skv freebsd_committer freebsd_triage 2008-12-12 17:34:54 UTC
State Changed
From-To: open->closed

Committed, thanks!