Re: Patch: BUG: H323 VoIP Calls not working

Cvetan Ivanov <[email protected]>
Newsgroups gmane.network.ethereal.devel
Message-ID <[email protected]>
Hello,

Thomas Sillaber wrote:
> Hola List,
> 
> the Voip Calls Feature is not working correctly with H.323 (SIP is working).
>  The Calls are not reassembled correctly. For every packet a new line is
> displayed. 
> 

It seems to me that in gtk/voip_calls.c tmp_h323info->guid is pointer 
itself, therefore:
memcmp(&tmp_h323info->guid
should in fact read:
memcmp(tmp_h323info->guid

Attached is a one line patch that seems to fix the h323 call graph for me.

Best regards,

Cvetan

-- 
Cvetan Ivanov
System Administrator
SpectrumNet Jsc.
+35929657613 Office

_______________________________________________
Ethereal-dev mailing list
[email protected]
http://www.ethereal.com/mailman/listinfo/ethereal-dev
voip_call_h323.diff (text/plain, 636 B)
Index: gtk/voip_calls.c
===================================================================
--- gtk/voip_calls.c	(revision 18189)
+++ gtk/voip_calls.c	(working copy)
@@ -1698,7 +1698,7 @@
 			tmp_listinfo=list->data;
 			if (tmp_listinfo->protocol == VOIP_H323){
 				tmp_h323info = tmp_listinfo->prot_info;
-				if ( (memcmp(&tmp_h323info->guid, &guid_allzero, GUID_LEN) != 0) && (memcmp(&tmp_h323info->guid, &pi->guid,GUID_LEN)==0) ){ 
+				if ( (memcmp(tmp_h323info->guid, &guid_allzero, GUID_LEN) != 0) && (memcmp(tmp_h323info->guid, &pi->guid,GUID_LEN)==0) ){ 
 					strinfo = (voip_calls_info_t*)(list->data);
 					break;
 				}
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.