Messages in this thread | ![/](/images/icornerl.gif) | | Date | Wed, 2 Feb 2011 19:38:27 +0200 | From | Gleb Natapov <> | Subject | Re: [PATCH] Fix EDD3.0 data verification. |
| |
On Wed, Feb 02, 2011 at 09:29:17AM -0800, H. Peter Anvin wrote: > On 02/02/2011 09:25 AM, Gleb Natapov wrote: > > According to spec length should be set to 30 on exit. > > I don't really know how this comment relates to the code, but in the > code I saw a fixed number of bytes being checksummed, which is > definitely wrong. > What length do you propose to use? If you were referring to params->length then it is incorrect since it will be always 30 on exit and this has nothing to do with size we need to checksum. Spec defines that the sum of bytes 30-73 should be zero. What is definitely wrong about this?
-- Gleb.
| ![\](/images/icornerr.gif) |