wotwot / haxevideo

Automatically exported from code.google.com/p/haxevideo
0 stars 0 forks source link

Maximum streams creation in one Connection is 12 #18

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Make a netconnect 
2. Make a netstream
3. Play a feed
4. close the netstream
5. Play a feed again
6. repeat 12 times.
7. 12th time results in error

What is the expected output? What do you see instead?
I expect the streams to keep opening.

Instead:

Processing Packet from Client 0
   header:{ kind => KCall, size => 25, channel => 3, src_dst => 0,
timestamp => 751 }
   packet: PCall(createStream,15,[ANull])
CREATESTREAM null
Client.hx:304: Assigning stream id 13
0   packet: PCommand(13,CClientBuffer(0))
Client.hx:508: COMMAND CClientBuffer(0):13
[2008-01-29 12:38:05] -
Called from server/RealtimeServer.hx line 121
Called from server/RealtimeServer.hx line 334
Called from server/Server.hx line 47
Called from server/Client.hx line 808
Called from format/Rtmp.hx line 373

What version of the product are you using? On what operating system?
1.0 Linux

Please provide any additional information below.

 A similar bug seems to have existed in Red5: http://jira.red5.org/browse/SN-14

Original issue reported on code.google.com by Nay...@gmail.com on 29 Jan 2008 at 6:39

GoogleCodeExporter commented 9 years ago
The reason for this happening is that the channels get assigned up to 64, 
because the
channel size is 6 bits. After 64 channels the channel bit is overflowed which 
causes
the entire readHeader function to get confused.

Original comment by Nay...@gmail.com on 29 Jan 2008 at 8:11

GoogleCodeExporter commented 9 years ago
Attached you'll find an attempt to solve this issue, based on the code to solve 
the
same bug in Red5, since I've been unable to find proper documentation of RTMP. 
At
least it seems to work for me :)

Original comment by alfredo....@gmail.com on 9 Mar 2008 at 11:55

Attachments:

GoogleCodeExporter commented 9 years ago
The code does not seems very accurate. Could you send me the link to the Red5 
SVN
sources at which place these changes were made ?

Original comment by ncanna...@gmail.com on 31 Aug 2008 at 3:32

GoogleCodeExporter commented 9 years ago
Ok, looking a bit more about it, it seems that it should be ok now. Thanks for 
reporting.

Original comment by ncanna...@gmail.com on 31 Aug 2008 at 4:30