Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#9197 - Return correct failure code if the WPS has too many viewports

Attached to Project: Rockbox
Opened by Alexander Levin (fml2) - Tuesday, 15 July 2008, 21:15 GMT+2
Last edited by Alexander Levin (fml2) - Monday, 11 October 2010, 16:56 GMT+2
Task Type Patches
Category Applications
Status Closed
Assigned To Nicolas Pennequin (nicolas_p)
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

The title says it all, I have nothing to add
   wps_parser.patch (0.5 KiB) 
 apps/gui/wps_parser.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

This task depends upon

Closed by  Alexander Levin (fml2)
Monday, 11 October 2010, 16:56 GMT+2
Reason for closing:  Rejected
Additional comments about closing:  A very old patch; obsoltete by now
Comment by Nicolas Pennequin (nicolas_p) - Wednesday, 16 July 2008, 01:11 GMT+2
Have you tested this patch? I don't think it will work correctly, because the value of PARSE_FAIL_LIMITS_EXCEEDED is positive, whereas parsing functions like parse_viewport are supposed to return either the number of characters they read or a strictly negative error value. Maybe there is no apparent problem (thanks to the other check for the viewport limit), but this looks wrong to me.
Comment by Alexander Levin (fml2) - Wednesday, 16 July 2008, 07:23 GMT+2
To be honest, I have not tested the patch. I just have seen that a wrong literal is used in this case and have assumed that the general frame is correctly set up. If it's not, then a fix is needed at another place (but this fix is still correct IMHO).
Comment by Alexander Levin (fml2) - Wednesday, 16 July 2008, 20:39 GMT+2
Here is another try. Now also tested, although not very extensively. Also added some comments.
   wps_parser2.patch (13.1 KiB) 
 apps/gui/wps_debug.c  |    2 
 apps/gui/gwps.h       |    9 ++-
 apps/gui/wps_parser.c |  120 ++++++++++++++++++++++++++++----------------------
 3 files changed, 76 insertions(+), 55 deletions(-)

Loading...