mainGNU ccRTP - Support: sr #103381, Need padding on/off fuction...

 
 

sr #103381: Need padding on/off fuction...

Submitter:  hjlee <bstill>
Submitted:  Tue 21 Sep 2004 05:09:48 AM UTC
   
 
Category:  None Priority:  5 - Normal
Severity:  1 - Wish Status:  Done
Privacy:  Public Assigned to:  None
Open/Closed:  Open Operating System:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 23 Mar 2005 10:08:41 AM UTC, comment #1: 

A new method (setPadding) has been added to outgoing queues. You
can set a variable padding length, and the default is 0 (padding
off).

Federico Montesino Pouzols <fedemp>
Group Member
Tue 21 Sep 2004 05:09:48 AM UTC, original submission:  

As I know, a rtp (outgoing) packet generated by ccRTP sets padding bit following code.
===============================
  uint8 padding = total & 0x03;
  if ( padding ) {
    padding = 4 - padding;
    total += padding;
  }
  // now we know the actual total length of the packet
  buffer = new unsigned char[total];
  (reinterpret_cast<uint32>(getHeader())) = 0;
  getHeader()->version = CCRTP_VERSION;
  if ( padding ) {
    memset(buffer + total - padding,0,padding - 1);
    buffer[total - 1] = padding;
    getHeader()->padding = 1;
  } else {
    getHeader()->padding = 0;
  }
===============================

It's seems to be that some Multimedia player, such as "freeamp", can't accept/play padded rtp packet stream.
I hope to be added a function to set padding flag by USER.


hjlee <bstill>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

Only logged-in users can vote.

 

Follows 1 latest change.

Date Changed by Updated Field Previous Value => Replaced by
2005-03-23 fedemp StatusNone Done

Back to the top

Powered by Savane 3.13-54b4.
Corresponding source code