List Info

Thread: Patch to player/src/codec/mpeg4/mpeg4.cpp




Patch to player/src/codec/mpeg4/mpeg4.cpp
user name
2007-03-13 15:43:00
It looks like the function parse_vovod is reading too far
into the
configuration string. Change line 134 from

      for (uint32_t ix = 0; ix < len; ix++) {

to

      for (uint32_t ix = 0; ix < len; ix += 2) {

fixes the problem. Seems too small to generate a patch.

HTH,
Pete



------------------------------------------------------------
-------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the
chance to share your
opinions on IT & business topics through brief
surveys-and earn cash
http://www.techsay.com/default.
php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Mpeg4ip-discussion mailing list
Mpeg4ip-discussionlists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mpeg
4ip-discussion

Re: Patch to player/src/codec/mpeg4/mpeg4.cpp
country flaguser name
United States
2007-03-13 16:08:36
Yes.  It shouldn't be too much a problem, however.  No
actual
harm or bugs result from it.  The buffers should be big
enough.

Bill

Peter J. Stieber wrote:
> It looks like the function parse_vovod is reading too
far into the
> configuration string. Change line 134 from
> 
>       for (uint32_t ix = 0; ix < len; ix++) {
> 
> to
> 
>       for (uint32_t ix = 0; ix < len; ix += 2) {
> 
> fixes the problem. Seems too small to generate a
patch.
> 
> HTH,
> Pete
> 
> 
> 
>
------------------------------------------------------------
-------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the
chance to share your
> opinions on IT & business topics through brief
surveys-and earn cash
> http://www.techsay.com/default.
php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> Mpeg4ip-discussion mailing list
> Mpeg4ip-discussionlists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/mpeg
4ip-discussion
> 

------------------------------------------------------------
-------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the
chance to share your
opinions on IT & business topics through brief
surveys-and earn cash
http://www.techsay.com/default.
php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Mpeg4ip-discussion mailing list
Mpeg4ip-discussionlists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mpeg
4ip-discussion

[1-2]

about | contact  Other archives ( Real Estate discussion Medical topics )