[Tcllib-bugs] [ tcllib-Bugs-2976290 ] decode fails

[SourceForge.net <no...@so...>]

Bugs item #2976290, was opened at 2010-03-24 21:45
Message generated for change (Comment added) made by andreas_kupries
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=112883&aid=2976290&group_id=12883

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: base64
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Alexander Danilov (daapp)
Assigned to: Andreas Kupries (andreas_kupries)
Summary: decode fails

Initial Comment:
This code:

::base64::decode s=GQMRAk5WXhsABh0NEx4RXBocBVgBHQMXHRgEFltMQENQXEFOExJVQ0RAQERUQ0dAEhYEExVIRRVVFENWKxMKABsPGBI6LRoYLhsEFhsXGFkXEwZXGQMIHg==

produce an error:
can't use empty string as operand of "&"
    while executing
"expr {($v << 2) | (($w & 0x30) >> 4)}"
    (procedure "::base64::decode" line 27)
    invoked from within
"::base64::decode s=GQMRAk5WXhsABh0NEx4RXBocBVgBHQMXHRgEFltMQENQXEFOExJVQ0RAQERUQ0dAEhYEExVIRRVVFENWKxMKABsPGBI6LRoYLhsEFhsXGFkXEwZXGQMIHg=="
    ("uplevel" body line 1)
    invoked from within
"uplevel #0 {::base64::decode s=GQMRAk5WXhsABh0NEx4RXBocBVgBHQMXHRgEFltMQENQXEFOExJVQ0RAQERUQ0dAEhYEExVIRRVVFENWKxMKABsPGBI6LRoYLhsEFhsXGFkXEwZXGQMIHg=..."

The source string may be invalid, but the code should not fail like this.


----------------------------------------------------------------------

>Comment By: Andreas Kupries (andreas_kupries)
Date: 2010-07-06 12:16

Message:
Changed code to detect the situation and throw a more properly looking
error ('Not enough data to process padding'). This is now version 2.4.2 of
base64.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=112883&aid=2976290&group_id=12883