Rockbox

Tasklist

FS#10496 - read_image close file?

Attached to Project: Rockbox
Opened by federico pelupessy (fip) - Saturday, 08 August 2009, 11:28 GMT
Last edited by Bertrik Sikken (bertrik) - Sunday, 09 August 2009, 22:21 GMT
Task Type Bugs
Category Utils
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Version 3.3
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

I am not sure about whether it is actually used somewhere, but shouldn't
read_image_file_offset also close the file?
This task depends upon

Closed by  Bertrik Sikken (bertrik)
Sunday, 09 August 2009, 22:21 GMT
Reason for closing:  Fixed
Additional comments about closing:  Function containing the bug has been removed in svn r22232, thanks for reporting.
Comment by Bertrik Sikken (bertrik) - Saturday, 08 August 2009, 15:15 GMT
I think your concern is valid, I can't see these files being closed anywhere.
Your patch will not work because the new code can not be reached.

The attached patch should fix that.
Comment by Bertrik Sikken (bertrik) - Saturday, 08 August 2009, 15:27 GMT
Hmm, on closer inspection, nobody seems to use that function so it's not causing any problems and we might just remove it altogether, see attached patch.
Maybe we should ask the original author first before removing.

Loading...