[asterisk-commits] russell: trunk r78185 - in /trunk: ./ include/asterisk/linkedlists.h

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Aug 6 11:51:31 CDT 2007


Author: russell
Date: Mon Aug  6 11:51:30 2007
New Revision: 78185

URL: http://svn.digium.com/view/asterisk?view=rev&rev=78185
Log:
Merged revisions 78184 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r78184 | russell | 2007-08-06 11:50:54 -0500 (Mon, 06 Aug 2007) | 5 lines

Fix the return value of AST_LIST_REMOVE().  This shouldn't be causing any
problems, though, because the only code that uses the return value only checks
to see if it is NULL.
(closes issue #10390, pointed out by mihai)

........

Modified:
    trunk/   (props changed)
    trunk/include/asterisk/linkedlists.h

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

Modified: trunk/include/asterisk/linkedlists.h
URL: http://svn.digium.com/view/asterisk/trunk/include/asterisk/linkedlists.h?view=diff&rev=78185&r1=78184&r2=78185
==============================================================================
--- trunk/include/asterisk/linkedlists.h (original)
+++ trunk/include/asterisk/linkedlists.h Mon Aug  6 11:51:30 2007
@@ -748,7 +748,7 @@
 		while (curelm && (curelm->field.next != (elm)))			\
 			curelm = curelm->field.next;			\
 		if (curelm) { \
-			__res = curelm; \
+			__res = (elm); \
 			curelm->field.next = (elm)->field.next;			\
 			if ((head)->last == (elm))				\
 				(head)->last = curelm;				\




More information about the asterisk-commits mailing list