byteman / rtsplib-java

Automatically exported from code.google.com/p/rtsplib-java
GNU Lesser General Public License v3.0
0 stars 0 forks source link

Error parsing Transport header when lower-transport parameter is present #3

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
There's a problem with the parsing of the Transport header when a 
lower-transport parameter is specified. The index increment at line 84 should 
be 3 instead of 2.

Excerpt from br/com/voicetechnology/rtspclient/headers/TransportHeader.java 
pinpointing the error:

                if(value.charAt(index) == '/')
                {
                        switch(value.charAt(++index))
                        {
                        case 'T':
                                transport = LowerTransport.TCP;
                                break;
                        case 'U':
                                transport = LowerTransport.UDP;
                                break;
                        default:
                                throw new IllegalArgumentException("Invalid Transport: "
                                                + value.substring(7));
                        }
                        index += 2;  // FIXME: Increment should be 3 instead of 2.
                } else
                        transport = LowerTransport.DEFAULT;

Original issue reported on code.google.com by flavios...@gmail.com on 12 Jan 2011 at 1:35

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r7.

Original comment by paulo.pa...@gmail.com on 21 Jan 2011 at 1:35