Rockbox

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

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#8683 - usb_core.c possible bug

Attached to Project: Rockbox
Opened by Joshua Simmons (mud) - Wednesday, 05 March 2008, 01:15 GMT+2
Last edited by Barry Wardell (barrywardell) - Wednesday, 12 March 2008, 21:18 GMT+2
Task Type Bugs
Category Drivers
Status Closed
Assigned To No-one
Player type All players
Severity Low
Priority Normal
Reported Version current build
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

In lines 477-481 of firmware/usbstack/usb_core.c (noticed in r16521) there is a possible bug. I'd make a patch but I don't know the correct fix.

The relevant text is:
if (req->wValue)
usb_drv_stall(req->wIndex & 0xf, false,(req->wIndex & 0x80) !=0);
else
usb_drv_stall(req->wIndex & 0xf, false,(req->wIndex & 0x80) !=0);
ack_control(req);


This is basically if(something) do_x(); else do_x(); do_y();
Very confusing at best.

I mentioned this in irc, but nobody seemed to notice. Sorry if someone is already looking into it.
This task depends upon

Closed by  Barry Wardell (barrywardell)
Wednesday, 12 March 2008, 21:18 GMT+2
Reason for closing:  Fixed
Comment by Frank Gevaerts (fg) - Wednesday, 12 March 2008, 17:41 GMT+2
Fixed. I saw it in the irc logs.

Loading...