Mailing-List: contact dev-help@activemq.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@activemq.apache.org
Message-ID: <934895962.13931265680414527.JavaMail.jira@brutus.apache.org>
Date: Tue, 9 Feb 2010 01:53:34 +0000 (UTC)
From: "yojay.li (JIRA)" <jira@apache.org>
To: dev@activemq.apache.org
Subject: [jira] Created: (AMQCPP-285) If brokerURI is ipv4
 format(127.0.0.1),  createConnection will throw
 decaf::lang::exceptions::NumberFormatException.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

If brokerURI is ipv4 format(127.0.0.1),  createConnection will throw decaf::lang::exceptions::NumberFormatException.
--------------------------------------------------------------------------------------------------------------------

                 Key: AMQCPP-285
                 URL: https://issues.apache.org/activemq/browse/AMQCPP-285
             Project: ActiveMQ C++ Client
          Issue Type: Bug
          Components: Decaf
    Affects Versions: 3.1
         Environment: vs2008,windows xp
            Reporter: yojay.li
            Assignee: Timothy Bish


file: 
  activemq-cpp-library-3.1.0\src\main\decaf\internal\net\URIHelper.cpp
funciton: 
  bool URIHelper::isValidIPv4Address( const std::string& host )
cause:
  The paramters of std::string.substr() is (start_pos, length) but not (start_pos, end_pos).
  Interger::parseInt(host.substr(..., ...)) will throw NumberFormatException.

fixed code:

bool URIHelper::isValidIPv4Address( const std::string& host ) {

    std::size_t index;
    std::size_t index2;

    try {

        int num;
        index = host.find( '.' );
		num = decaf::lang::Integer::parseInt( host.substr( 0, index ) );
        if( num < 0 || num > 255 ) {
            return false;
        }

        index2 = host.find( '.', index + 1 );
        num = decaf::lang::Integer::parseInt( host.substr( index + 1, index2 - index - 1) );
        if( num < 0 || num > 255 ) {
            return false;
        }

        index = host.find( '.', index2 + 1 );
        num = decaf::lang::Integer::parseInt( host.substr( index2 + 1, index - index2 - 1) );
        if( num < 0 || num > 255 ) {
            return false;
        }

        num = decaf::lang::Integer::parseInt( host.substr( index + 1, std::string::npos ) );
        if( num < 0 || num > 255 ) {
            return false;
        }

	} catch( decaf::lang::Exception& e ) {
        return false;
    }

    return true;
}
  

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.