From: SourceForge.net <no...@so...> - 2011-08-09 21:51:01
|
Bugs item #3388285, was opened at 2011-08-08 13:41 Message generated for change (Comment added) made by sbaldovi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=596648&aid=3388285&group_id=91293 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Sergio Baldovi (sbaldovi) Assigned to: Nobody/Anonymous (nobody) Summary: libdisk, memset parameters likely transposed Initial Comment: I think there is another transposed memset in libdisk. Could anyone please check this patch? ---------------------------------------------------------------------- >Comment By: Sergio Baldovi (sbaldovi) Date: 2011-08-09 23:51 Message: Thanks for looking at this. Unfortunately, I can't test this right now because my knowledge of disk structures and disk usage is very limited. Seems a bit challenging, though. ---------------------------------------------------------------------- Comment By: Fredrick Meunier (fredm) Date: 2011-08-09 16:22 Message: The patch looks right to me and matches the similar fill data elsewhere in the file (e.g. memset( d->track + 1, autofill & 0xff, d->bpt ); ). Looks like you would need to test it with a UDI image to be completely sure the results were right. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=596648&aid=3388285&group_id=91293 |