From 2525748962c69c005b3c73779b26c8883b219956 Mon Sep 17 00:00:00 2001 From: Philip Hazel Date: Mon, 3 Jul 2006 15:19:44 +0000 Subject: Patch to use select() when poll() not available in spam.c. --- doc/doc-txt/ChangeLog | 5 ++++- src/ACKNOWLEDGMENTS | 6 +++--- src/src/spam.c | 23 +++++++++++++++++++---- 3 files changed, 26 insertions(+), 8 deletions(-) diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog index 79ce6d7d7..869a9c270 100644 --- a/doc/doc-txt/ChangeLog +++ b/doc/doc-txt/ChangeLog @@ -1,4 +1,4 @@ -$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.365 2006/06/30 15:36:08 ph10 Exp $ +$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.366 2006/07/03 15:19:44 ph10 Exp $ Change log file for Exim from version 4.21 ------------------------------------------- @@ -67,6 +67,9 @@ PH/08 When testing addresses using -bt, indicate those final addresses that (Suppressing duplicates isn't a good idea as you lose the information about possibly different redirections that led to the duplicates.) +PH/09 Applied patch from Erik to use select() instead of poll() in spam.c on + systems where poll() doesn't work, in particular OS X. + Exim version 4.62 ----------------- diff --git a/src/ACKNOWLEDGMENTS b/src/ACKNOWLEDGMENTS index 114cc64c9..4090d2409 100644 --- a/src/ACKNOWLEDGMENTS +++ b/src/ACKNOWLEDGMENTS @@ -1,4 +1,4 @@ -$Cambridge: exim/src/ACKNOWLEDGMENTS,v 1.49 2006/06/28 16:00:23 ph10 Exp $ +$Cambridge: exim/src/ACKNOWLEDGMENTS,v 1.50 2006/07/03 15:19:44 ph10 Exp $ EXIM ACKNOWLEDGEMENTS @@ -20,7 +20,7 @@ relatively small patches. Philip Hazel Lists created: 20 November 2002 -Last updated: 28 June 2006 +Last updated: 03 July 2006 THE OLD LIST @@ -253,5 +253,5 @@ Stephen Wilcox Patch for ignore_enotdir problem Alain Williams Suggested patch for exicyclog options David Woodhouse SQLite support proof of concept code control=freeze/no_tell basic code - +Erik ? patch to use select() instead of poll() on OS X **** diff --git a/src/src/spam.c b/src/src/spam.c index e49bc56bc..25f724236 100644 --- a/src/src/spam.c +++ b/src/src/spam.c @@ -1,4 +1,4 @@ -/* $Cambridge: exim/src/src/spam.c,v 1.11 2005/08/01 14:41:25 ph10 Exp $ */ +/* $Cambridge: exim/src/src/spam.c,v 1.12 2006/07/03 15:19:44 ph10 Exp $ */ /************************************************* * Exim - an Internet mail transport agent * @@ -42,6 +42,9 @@ int spam(uschar **listptr) { struct sockaddr_un server; #ifndef NO_POLL_H struct pollfd pollfd; +#else /* Patch posted by Erik ? for OS X */ + struct timeval select_tv; /* and applied by PH */ + fd_set select_fd; #endif /* stop compiler warning */ @@ -218,7 +221,8 @@ int spam(uschar **listptr) { * and we poll the desciptor to make sure that we can write without * blocking. Short writes are gracefully handled and if the whole * trasaction takes too long it is aborted. - * Note: poll() is not supported in OSX 10.2. + * Note: poll() is not supported in OSX 10.2 and is reported to be + * broken in more recent versions (up to 10.4). */ #ifndef NO_POLL_H pollfd.fd = spamd_sock; @@ -232,8 +236,19 @@ int spam(uschar **listptr) { again: #ifndef NO_POLL_H result = poll(&pollfd, 1, 1000); + +/* Patch posted by Erik ? for OS X and applied by PH */ +#else + select_tv.tv_sec = 1; + select_tv.tv_usec = 0; + FD_ZERO(&select_fd); + FD_SET(spamd_sock, &select_fd); + result = select(spamd_sock+1, NULL, &select_fd, NULL, &select_tv); +#endif +/* End Erik's patch */ + if (result == -1 && errno == EINTR) - continue; + goto again; else if (result < 1) { if (result == -1) log_write(0, LOG_MAIN|LOG_PANIC, @@ -248,7 +263,7 @@ again: (void)fclose(mbox_file); return DEFER; } -#endif + wrote = send(spamd_sock,spamd_buffer + offset,read - offset,0); if (wrote == -1) { -- cgit v1.2.3