the latest git tree fro yesterday + the latest git tree of cppcheck yields into 
these warning :


The 2 "if"s seems to be superfluous, or ? :

    if (iplace == sizeof iconvert)
        iplace--;
    iconvert[iplace] = 0;<--- Array 'iconvert[20]' accessed at index 20, which 
is out of bounds. Otherwise condition 'iplace==20' is redundant.

    /* convert fractional part */
    do {
        fconvert[fplace++] =
            (caps ? "0123456789ABCDEF"
              : "0123456789abcdef")[fracpart % 10];
        fracpart = (fracpart / 10);
    } while (fplace < max);
    if (fplace == sizeof fconvert)
        fplace--;
    fconvert[fplace] = 0;<--- Array 'fconvert[20]' accessed at index 20, which 
is out of bounds. Otherwise condition 'fplace==20' is redundant.


-- 
Toralf

______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [email protected]

Reply via email to