[asterisk-commits] bbryant: branch 1.4 r72383 - in /branches/1.4: ./ main/asterisk.c

[SVN commits to the Asterisk project]

Author: bbryant
Date: Wed Jun 27 18:29:14 2007
New Revision: 72383

URL: http://svn.digium.com/view/asterisk?view=rev&rev=72383
Log:
Merged revisions 72373 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r72373 | bbryant | 2007-06-27 18:22:13 -0500 (Wed, 27 Jun 2007) | 3 lines

Reinstating patch. This actually fixes the problem, however I was running a development branch without it and mistakenly thought it wasn't fixed. 
Fixes issue #10010, and #9654: 100% CPU usage caused by an asterisk console losing it's controlling terminal.

........

Modified:
    branches/1.4/   (props changed)
    branches/1.4/main/asterisk.c

Propchange: branches/1.4/
------------------------------------------------------------------------------
Binary property 'branch-1.2-merged' - no diff available.

Modified: branches/1.4/main/asterisk.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/main/asterisk.c?view=diff&rev=72383&r1=72382&r2=72383
==============================================================================
--- branches/1.4/main/asterisk.c (original)
+++ branches/1.4/main/asterisk.c Wed Jun 27 18:29:14 2007
@@ -2282,6 +2282,9 @@
 	for (;;) {
 		ebuf = (char *)el_gets(el, &num);
 
+		if (!ebuf && write(1, "", 1) < 0)
+			break;
+
 		if (!ast_strlen_zero(ebuf)) {
 			if (ebuf[strlen(ebuf)-1] == '\n')
 				ebuf[strlen(ebuf)-1] = '\0';
@@ -2957,6 +2960,9 @@
 		for (;;) {
 			buf = (char *)el_gets(el, &num);
 
+			if (!buf && write(1, "", 1) < 0)
+				goto lostterm;
+
 			if (buf) {
 				if (buf[strlen(buf)-1] == '\n')
 					buf[strlen(buf)-1] = '\0';
@@ -2979,5 +2985,6 @@
 
 	monitor_sig_flags(NULL);
 
+lostterm:
 	return 0;
 }