[bugs #6849] DigestInputStream computes the wrong result

[Mark Wielaard]

This mail is an automated notification from the bugs tracker
 of the project: classpath.

/**************************************************************************/
[bugs #6849] Latest Modifications:

Changes by: 
                Mark Wielaard <address@hidden>
'Date: 
                Sat 11/29/03 at 00:03 (Europe/Amsterdam)

            What     | Removed                   | Added
---------------------------------------------------------------------------
          Resolution | None                      | Fixed
         Assigned to | None                      | mark
              Status | Open                      | Closed


------------------ Additional Follow-up Comments ----------------------------
Thanks a lot. This is obviously correct.

I committed the patch as follows:



2003-11-28  Mark Wielaard  <address@hidden>

                                                                                

        Reported by Archie Cobbs:

        * java/security/DigestInputStream.java (read(byte[], int, int): Call

        digest.update() with temp, not len as lenght.



Will turn the test case into a mauve test.






/**************************************************************************/
[bugs #6849] Full Item Snapshot:

URL: <http://savannah.gnu.org/bugs/?func=detailitem&item_id=6849>
Project: classpath
Submitted by: Archie Cobbs
On: Fri 11/28/03 at 20:45

Severity:  5 - Average
Resolution:  Fixed
Assigned to:  mark
Status:  Closed
Platform Version:  None


Summary:  DigestInputStream computes the wrong result

Original Submission:  DigestInputStream passes extra bytes through the

MessageDigest, causing the wrong hash value to be

computed. The bug is obvious once you look at the

erroneous code.. see attached patch.

Follow-up Comments
------------------


-------------------------------------------------------
Date: Sat 11/29/03 at 00:03         By: mark
Thanks a lot. This is obviously correct.

I committed the patch as follows:



2003-11-28  Mark Wielaard  <address@hidden>

                                                                                

        Reported by Archie Cobbs:

        * java/security/DigestInputStream.java (read(byte[], int, int): Call

        digest.update() with temp, not len as lenght.



Will turn the test case into a mauve test.






File Attachments
-------------------

-------------------------------------------------------
Date: Fri 11/28/03 at 20:57  Name: MD5.java  Size: 652KB   By: archie172
Test case
http://savannah.gnu.org/bugs/download.php?item_id=6849&amp;item_file_id=885

-------------------------------------------------------
Date: Fri 11/28/03 at 20:45  Name: dis.patch  Size: 786KB   By: archie172
Patch to fix this bug
http://savannah.gnu.org/bugs/download.php?item_id=6849&amp;item_file_id=884






For detailed info, follow this link:
<http://savannah.gnu.org/bugs/?func=detailitem&item_id=6849>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/