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;
}