Re: [streaming] miniSAP

Rémi Denis-Courmont <[email protected]>
Newsgroups gmane.comp.video.videolan.vls.general
Organization Remlab.net
Message-ID <[email protected]>
	Hello,

Le lundi 30 juin 2008 19:52:27 Marshall Eubanks, vous avez écrit :
> On Jun 30, 2008, at 12:34 PM, Rémi Denis-Courmont wrote:
> > Le dimanche 29 juin 2008 22:18:55 Marshall Eubanks, vous avez écrit :
> >>> In other words, I had put in two ports, but miniSAP combined them
> >>> into one "video port 5432534 udp 33"
> >>> Not sure if that was the problem.
> >
> > You're taking minisap way beyond its intended use.
>
> No doubt, but the issue to me is that the software, if misused, does
> bad things. Can this be prevented ?

Sure. I think I found the problem, but it has nothing to do with port numbers.

-- 
Rémi Denis-Courmont
http://www.remlab.net/

_______________________________________________
streaming mailing list
To unsubscribe or modify your subscription options:
http://mailman.videolan.org/listinfo/streaming
minisap.patch (text/x-diff, 485 B)
Index: program.cpp
===================================================================
--- program.cpp	(révision 339)
+++ program.cpp	(révision 340)
@@ -69,10 +69,9 @@
 void Program::SetHasPlGroup(bool b){b_has_pl_group = b ;}
 void Program::SetPort(const char* p)
 {
-    int i_port=atoi(p);
     char psz_port[6];
-    snprintf(psz_port, sizeof (psz_port), "%i",i_port);
-    port +=psz_port;
+    snprintf(psz_port, sizeof (psz_port), "%d", atoi(p));
+    port = psz_port;
 
 }
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.