From aeh at db.org Mon Jan 12 02:44:19 2009 From: aeh at db.org (Alfred E. Heggestad) Date: Mon, 12 Jan 2009 11:44:19 +0100 Subject: [RTPproxy Devel] CVS:commitlog: rtpproxy rtpp_defines.h In-Reply-To: <20081224102806.DBE991511A5@mail.berlios.de> References: <20081224102806.DBE991511A5@mail.berlios.de> Message-ID: <496B1F03.505@db.org> Hi Maxim I think this patch broke the build on linux: In file included from rtpp_session.c:41: rtpp_defines.h:120: error: expected specifier-qualifier-list before ?uint8_t? rtpp_session.c: In function ?init_hash_table?: rtpp_session.c:53: error: ?struct cfg? has no member named ?rand_table? rtpp_session.c: In function ?hash_string?: rtpp_session.c:62: error: ?struct cfg? has no member named ?rand_table? rtpp_session.c:63: error: ?struct cfg? has no member named ?rand_table? rtpp_session.c: In function ?hash_table_append?: rtpp_session.c:78: error: ?struct cfg? has no member named ?hash_table? rtpp_session.c:80: error: ?struct cfg? has no member named ?hash_table? rtpp_session.c: In function ?hash_table_remove?: rtpp_session.c:108: error: ?struct cfg? has no member named ?hash_table? rtpp_session.c:109: error: ?struct cfg? has no member named ?hash_table? rtpp_session.c: In function ?session_findfirst?: rtpp_session.c:122: error: ?struct cfg? has no member named ?hash_table? rtpp_session.c: In function ?remove_session?: rtpp_session.c:164: error: ?struct cfg? has no member named ?log_level? rtpp_session.c:167: error: ?struct cfg? has no member named ?log_level? rtpp_session.c:170: error: ?struct cfg? has no member named ?log_level? rtpp_session.c: In function ?reconnect_timeout_handler?: rtpp_session.c:285: error: ?struct cfg? has no member named ?log_level? rtpp_session.c:287: error: ?struct cfg? has no member named ?log_level? rtpp_session.c:292: error: ?struct cfg? has no member named ?log_level? rtpp_session.c:302: error: ?struct cfg? has no member named ?log_level? rtpp_session.c: In function ?do_timeout_notification?: rtpp_session.c:322: error: ?struct cfg? has no member named ?log_level? rtpp_session.c:342: error: ?struct cfg? has no member named ?log_level? make[1]: *** [rtpp_session.o] Error 1 make[1]: Leaving directory `/home/alfredh/cvs/rtpproxy' make: *** [all] Error 2 (Build host is Debian Lenny, gcc v4.3.2) one way to fix this, is to include config.h from rtpp_defines.h: Index: rtpp_defines.h =================================================================== RCS file: /cvsroot/ser/rtpproxy/rtpp_defines.h,v retrieving revision 1.23 diff -u -3 -r1.23 rtpp_defines.h --- rtpp_defines.h 24 Dec 2008 10:28:06 -0000 1.23 +++ rtpp_defines.h 12 Jan 2009 10:42:02 -0000 @@ -31,6 +31,8 @@ #ifndef _RTPP_DEFINES_H_ #define _RTPP_DEFINES_H_ +#include "config.h" + #include #include #include this fixes the build error, and if this is a good solution, then I think you should make sure that config.h is included in all .[hc] files. Then, there is only one warning left: $ make -s rtpp_command.c: In function ?handle_command?: rtpp_command.c:510: warning: pointer targets in passing argument 1 of ?url_unquote? differ in signedness /alfred Maxim Sobolev wrote: > sobomax 2008/12/24 11:28:06 CET > > SER CVS Repository > > Modified files: > . rtpp_defines.h > Log: > Use HAVE_STDINT_H instead of checking for particular operating system. > > Revision Changes Path > 1.23 +2 -4 rtpproxy/rtpp_defines.h > http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtpp_defines.h.diff?r1=1.22&r2=1.23 > _______________________________________________ > Devel mailing list > Devel at rtpproxy.org > http://lists.rtpproxy.org/mailman/listinfo/devel From sobomax at sippysoft.com Mon Jan 12 02:55:28 2009 From: sobomax at sippysoft.com (Maxim Sobolev) Date: Mon, 12 Jan 2009 11:55:28 +0100 (CET) Subject: [RTPproxy Devel] CVS:commitlog: rtpproxy rtpp_defines.h Message-ID: <20090112105528.0117F16837E@mail.berlios.de> sobomax 2009/01/12 11:55:28 CET SER CVS Repository Modified files: . rtpp_defines.h Log: Include config.h, we need it before we can test for HAVE_STDINT_H. Reminded by: Alfred E. Heggestad Revision Changes Path 1.24 +3 -1 rtpproxy/rtpp_defines.h http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtpp_defines.h.diff?r1=1.23&r2=1.24 From sobomax at sippysoft.com Mon Jan 12 02:56:59 2009 From: sobomax at sippysoft.com (Maxim Sobolev) Date: Mon, 12 Jan 2009 11:56:59 +0100 (CET) Subject: [RTPproxy Devel] CVS:commitlog: rtpproxy rtpp_command.c Message-ID: <20090112105659.7542216837E@mail.berlios.de> sobomax 2009/01/12 11:56:59 CET SER CVS Repository Modified files: . rtpp_command.c Log: Cast argument of url_unquote() so that gcc doesn't warn about difference in signess. Revision Changes Path 1.24 +2 -2 rtpproxy/rtpp_command.c http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtpp_command.c.diff?r1=1.23&r2=1.24 From sobomax at sippysoft.com Mon Jan 12 03:36:41 2009 From: sobomax at sippysoft.com (Maxim Sobolev) Date: Mon, 12 Jan 2009 12:36:41 +0100 (CET) Subject: [RTPproxy Devel] CVS:commitlog: rtpproxy rtp.c rtp.h rtp_resizer.c Message-ID: <20090112113641.6BC22F4282@mail.berlios.de> sobomax 2009/01/12 12:36:41 CET SER CVS Repository Modified files: . rtp.c rtp.h rtp_resizer.c Log: Extend rtp_packet_parse() function to perform better validation of all fields of the RTP packets (based on the RFC1889) and return error code if any fields is out of bound. For now, this is only relevant when re-packetization feature is active, for regular forwarding the RTPproxy doesn't perform any parsing/validation. Revision Changes Path 1.9 +71 -8 rtpproxy/rtp.c http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtp.c.diff?r1=1.8&r2=1.9 1.11 +20 -2 rtpproxy/rtp.h http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtp.h.diff?r1=1.10&r2=1.11 1.6 +3 -2 rtpproxy/rtp_resizer.c http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtp_resizer.c.diff?r1=1.5&r2=1.6 From sobomax at sippysoft.com Mon Jan 12 03:37:40 2009 From: sobomax at sippysoft.com (Maxim Sobolev) Date: Mon, 12 Jan 2009 12:37:40 +0100 (CET) Subject: [RTPproxy Devel] CVS:commitlog: rtpproxy configure.ac Message-ID: <20090112113740.062C8F4282@mail.berlios.de> sobomax 2009/01/12 12:37:39 CET SER CVS Repository Modified files: . configure.ac Log: Bump version to 200901120. Revision Changes Path 1.26 +2 -2 rtpproxy/configure.ac http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/configure.ac.diff?r1=1.25&r2=1.26 From sobomax at sippysoft.com Mon Jan 12 03:39:12 2009 From: sobomax at sippysoft.com (Maxim Sobolev) Date: Mon, 12 Jan 2009 12:39:12 +0100 (CET) Subject: [RTPproxy Devel] CVS:commitlog: rtpproxy configure.ac Message-ID: <20090112113912.178F2F4282@mail.berlios.de> sobomax 2009/01/12 12:39:12 CET SER CVS Repository Modified files: . configure.ac Log: Upgrade from alpha to beta. Revision Changes Path 1.27 +2 -2 rtpproxy/configure.ac http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/configure.ac.diff?r1=1.26&r2=1.27 From sobomax at sippysoft.com Mon Jan 12 03:40:14 2009 From: sobomax at sippysoft.com (Maxim Sobolev) Date: Mon, 12 Jan 2009 12:40:14 +0100 (CET) Subject: [RTPproxy Devel] CVS:commitlog: rtpproxy configure Message-ID: <20090112114014.6C5DC154248@mail.berlios.de> sobomax 2009/01/12 12:40:14 CET SER CVS Repository Modified files: . configure Log: Re-gen. Revision Changes Path 1.22 +10 -10 rtpproxy/configure http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/configure.diff?r1=1.21&r2=1.22 From sobomax at sippysoft.com Mon Jan 12 03:49:09 2009 From: sobomax at sippysoft.com (Maxim Sobolev) Date: Mon, 12 Jan 2009 03:49:09 -0800 Subject: [RTPproxy Devel] CVS:commitlog: rtpproxy rtpp_defines.h In-Reply-To: <496B1F03.505@db.org> References: <20081224102806.DBE991511A5@mail.berlios.de> <496B1F03.505@db.org> Message-ID: <496B2E35.8040003@sippysoft.com> Alfred E. Heggestad wrote: > Hi Maxim > > I think this patch broke the build on linux: Alfred, Thank you for letting me know. I had noticed this problem quite some time ago, and fixed it in our private version but forgot to check in fix into public tree. I've fixed the problem and updated 1.2 snapshot on the www.rtpproxy.org page as well. Regards, -- Maksym Sobolyev Sippy Software, Inc. Internet Telephony (VoIP) Experts T/F: +1-646-651-1110 Web: http://www.sippysoft.com MSN: sales at sippysoft.com Skype: SippySoft From aeh at db.org Mon Jan 12 04:38:41 2009 From: aeh at db.org (Alfred E. Heggestad) Date: Mon, 12 Jan 2009 13:38:41 +0100 Subject: [RTPproxy Devel] [PATCH] Update RPM spec file to v1.2 Message-ID: <496B39D1.20603@db.org> Index: rpm/rtpproxy.spec =================================================================== RCS file: /cvsroot/ser/rtpproxy/rpm/rtpproxy.spec,v retrieving revision 1.1 diff -u -3 -r1.1 rtpproxy.spec --- rpm/rtpproxy.spec 22 Feb 2007 14:49:57 -0000 1.1 +++ rpm/rtpproxy.spec 12 Jan 2009 12:39:13 -0000 @@ -1,6 +1,6 @@ %define name rtpproxy -%define ver 0.3 -%define rel 1 +%define ver 1.2 +%define rel beta.200901120 Name: %name Version: %ver @@ -50,11 +50,15 @@ %files %defattr(644,root,root,755) %doc AUTHORS COPYING README +%{_mandir}/man8/* %attr(755,root,root) %{_bindir}/rtpproxy %attr(755,root,root) %{_bindir}/makeann -%attr(755,root,root) /etc/rc.d/init.d/* +%config %attr(755,root,root) /etc/rc.d/init.d/* %changelog +* Mon Jan 12 2009 Alfred E. Heggestad +- Updated for version 1.2 + * Tue Jan 30 2007 Alfred E. Heggestad - 0.3.1 - Initial build.