[asterisk-commits] mnicholson: trunk r269083 - /trunk/apps/app_fax.c

[SVN commits to the Asterisk project]

Author: mnicholson
Date: Tue Jun  8 13:50:45 2010
New Revision: 269083

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=269083
Log:
Don't pass null to manager_event()

(closes issue #17087)
Reported by: bklang
Patches:
      app-fax-null-sprintf1.diff uploaded by mnicholson (license 96)
Tested by: bklang


Modified:
    trunk/apps/app_fax.c

Modified: trunk/apps/app_fax.c
URL: http://svnview.digium.com/svn/asterisk/trunk/apps/app_fax.c?view=diff&rev=269083&r1=269082&r2=269083
==============================================================================
--- trunk/apps/app_fax.c (original)
+++ trunk/apps/app_fax.c Tue Jun  8 13:50:45 2010
@@ -227,8 +227,8 @@
 	
 	s->finished = 1; 
 	
-	local_ident = t30_get_tx_ident(f);
-	far_ident = t30_get_rx_ident(f);
+	local_ident = S_OR(t30_get_tx_ident(f), "");
+	far_ident = S_OR(t30_get_rx_ident(f), "");
 	pbx_builtin_setvar_helper(s->chan, "FAXSTATUS", "SUCCESS"); 
 	pbx_builtin_setvar_helper(s->chan, "FAXERROR", NULL); 
 	pbx_builtin_setvar_helper(s->chan, "REMOTESTATIONID", far_ident);