<HTML><HEAD>
<META http-equiv=Content-Type content="text/html; charset=iso-8859-1">
<META content="IncrediMail 1.0" name=GENERATOR></HEAD>
<BODY style="BACKGROUND-POSITION: 0px 0px; FONT-SIZE: 12pt; MARGIN: 5px 10px 10px; FONT-FAMILY: Arial" bgColor=#ffffff background="" scroll=yes ORGYPOS="0">
<TABLE id=INCREDIMAINTABLE cellSpacing=0 cellPadding=2 width="100%" border=0>
<TBODY>
<TR>
<TD id=INCREDITEXTREGION style="FONT-SIZE: 12pt; CURSOR: auto; FONT-FAMILY: Arial" width="100%">
<DIV>Hi Chih-Wei,</DIV>
<DIV> </DIV>
<DIV>Still having this error eventhough I have patched chan_h323.c & others.</DIV>
<DIV> </DIV>
<DIV> </DIV>
<DIV>Rgds</DIV>
<DIV>T.E.Yap</DIV>
<DIV> </DIV>
<DIV> </DIV>
<DIV> PWLib version is 1.8.1... ok<BR>OpenH323 version is 1.15.1... ok<BR>g++ -g -c -o ast_h323.o -march=i686 -DPBYTE_ORDER=PLITTLE_ENDIAN -DNDEBUG -DDO_CRASH -DDEBUG_THREADS -pip<BR>e -Wall -fPIC -Wmissing-prototypes -D_REENTRANT -D_GNU_SOURCE -I../../include -I/home/teyap/pwlib/include<BR> -I/home/teyap/openh323/include -Wno-missing-prototypes ast_h323.cpp<BR>cc1plus: warning: command line option "-Wmissing-prototypes" is valid for Ada/C/ObjC but not for C++<BR>cc1plus: warning: command line option "-Wno-missing-prototypes" is valid for Ada/C/ObjC but not for C++<BR>ast_h323.cpp: In constructor `MyH323_ExternalRTPChannel::MyH323_ExternalRTPChannel(MyH323Connection&, con<BR>st H323Capability&, H323Channel::Directions, unsigned int)':<BR>ast_h323.cpp:940: error: 'struct rtp_info' has no member named 'addr'<BR>ast_h323.cpp:941: error: 'struct rtp_info' has no member named 'port'<BR>make: *** [ast_h323.o] Error 1</DIV>
<DIV> </DIV>
<DIV id=IncrediOriginalMessage><I>-------Original Message-------</I></DIV>
<DIV> </DIV>
<DIV id=receivestrings>
<DIV dir=ltr style="FONT-SIZE: 11pt"><I><B>From:</B></I> <A href="mailto:cwhuang@citron.com.tw">Chih-Wei Huang</A></DIV>
<DIV dir=ltr style="FONT-SIZE: 11pt"><I><B>Date:</B></I> 03/24/05 09:30:31</DIV>
<DIV dir=ltr style="FONT-SIZE: 11pt"><I><B>To:</B></I> <A href="mailto:asterisk-dev@lists.digium.com">Asterisk Developers Mailing List</A></DIV>
<DIV dir=ltr style="FONT-SIZE: 11pt"><I><B>Subject:</B></I> Re: [Asterisk-Dev] Re: Asterisk H.323 driver</DIV></DIV>
<DIV> </DIV>
<DIV>Charles Wang wrote:</DIV>
<DIV>></DIV>
<DIV>> I guess that you foget to patch the chan_h323.c under "channels/" too.</DIV>
<DIV>> Such as:</DIV>
<DIV>></DIV>
<DIV>> channels/chan_h323.c lines 1179 to 1181</DIV>
<DIV>> info->addr = ast_inet_ntoa(iabuf, sizeof(iabuf), bindaddr.sin_addr);</DIV>
<DIV>> info->port = ntohs(us.sin_port);</DIV>
<DIV>> ast_log(LOG_DEBUG, "Sending RTP 'US' %s:%d\n", info->addr, info->port);</DIV>
<DIV> </DIV>
<DIV>Yes, you are right.</DIV>
<DIV>I forgot the changes in chan_h323.c. Very sorry!</DIV>
<DIV> </DIV>
<DIV>Here it is:</DIV>
<DIV> </DIV>
<DIV>Index: chan_h323.c</DIV>
<DIV>===================================================================</DIV>
<DIV>RCS file: /usr/cvsroot/asterisk/channels/chan_h323.c,v</DIV>
<DIV>retrieving revision 1.106</DIV>
<DIV>diff -u -r1.106 chan_h323.c</DIV>
<DIV>--- chan_h323.c 17 Mar 2005 21:30:19 -0000 1.106</DIV>
<DIV>+++ chan_h323.c 24 Mar 2005 01:25:58 -0000</DIV>
<DIV>@@ -1159,9 +1160,7 @@</DIV>
<DIV> struct rtp_info *external_rtp_create(unsigned call_reference, const</DIV>
<DIV>char * token)</DIV>
<DIV> {</DIV>
<DIV> struct oh323_pvt *pvt;</DIV>
<DIV>- struct sockaddr_in us;</DIV>
<DIV> struct rtp_info *info;</DIV>
<DIV>- static char iabuf[INET_ADDRSTRLEN];</DIV>
<DIV> </DIV>
<DIV> info = (struct rtp_info *)malloc(sizeof(struct rtp_info));</DIV>
<DIV> if (!info) {</DIV>
<DIV>@@ -1174,11 +1173,8 @@</DIV>
<DIV> return NULL;</DIV>
<DIV> }</DIV>
<DIV> /* figure out our local RTP port and tell the H.323 stack about</DIV>
<DIV>it */</DIV>
<DIV>- ast_rtp_get_us(pvt->rtp, &us);</DIV>
<DIV>- /* evil hack, until I (or someone?) figures out a better way */</DIV>
<DIV>- info->addr = ast_inet_ntoa(iabuf, sizeof(iabuf), bindaddr.sin_addr);</DIV>
<DIV>- info->port = ntohs(us.sin_port);</DIV>
<DIV>- ast_log(LOG_DEBUG, "Sending RTP 'US' %s:%d\n", info->addr,</DIV>
<DIV>info->port);</DIV>
<DIV>+ ast_rtp_get_us(pvt->rtp, &info->rtp_addr);</DIV>
<DIV>+ info->rtp_addr.sin_addr = bindaddr.sin_addr; /* hmm... do we</DIV>
<DIV>need this? */</DIV>
<DIV> return info;</DIV>
<DIV> }</DIV>
<DIV> </DIV>
<DIV> </DIV>
<DIV> </DIV>
<DIV>--</DIV>
<DIV> ~ Chih-Wei Huang (<A href="mailto:cwhuang@citron.com.tw">cwhuang@citron.com.tw</A>)</DIV>
<DIV> 'v' CTO, Citron Network Inc. ( <A href="http://www.citron.com.tw/">http://www.citron.com.tw/</A> )</DIV>
<DIV> // \\ GnuGK Project : <A href="http://www.gnugk.org/">http://www.gnugk.org/</A> (Developer)</DIV>
<DIV>/( )\ HomePage : <A href="http://www.linux.org.tw/~cwhuang/">http://www.linux.org.tw/~cwhuang/</A></DIV>
<DIV> ^`~'^</DIV>
<DIV>_______________________________________________</DIV>
<DIV>Asterisk-Dev mailing list</DIV>
<DIV><A href="mailto:Asterisk-Dev@lists.digium.com">Asterisk-Dev@lists.digium.com</A></DIV>
<DIV><A href="http://lists.digium.com/mailman/listinfo/asterisk-dev">http://lists.digium.com/mailman/listinfo/asterisk-dev</A></DIV>
<DIV>To UNSUBSCRIBE or update options visit:</DIV>
<DIV> <A href="http://lists.digium.com/mailman/listinfo/asterisk-dev">http://lists.digium.com/mailman/listinfo/asterisk-dev</A></DIV></TD></TR>
<TR>
<TD id=INCREDIFOOTER width="100%">
<TABLE cellSpacing=0 cellPadding=0 width="100%">
<TBODY>
<TR>
<TD width="100%"></TD>
<TD id=INCREDISOUND vAlign=bottom align=middle></TD>
<TD id=INCREDIANIM vAlign=bottom align=middle></TD></TR></TBODY></TABLE></TD></TR></TBODY></TABLE><SPAN id=IncrediStamp><SPAN ltr??><A href="http://www.incredimail.com/index.asp?id=409&lang=9"><IMG alt="" hspace=0 src="cid:776405DE-D2CA-436E-B29A-0E53BA5168FA" align=baseline border=0></A></SPAN></SPAN></BODY></HTML>