Comment 4 for bug 956967

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (milestone-proposed)

Reviewed: https://review.openstack.org/5776
Committed: http://github.com/openstack/nova/commit/bacc688897047b06df15326b67d4130ce706604e
Submitter: Jenkins
Branch: milestone-proposed

commit bacc688897047b06df15326b67d4130ce706604e
Author: Greg Althaus <email address hidden>
Date: Fri Mar 16 06:41:54 2012 -0700

    Tweak security port validation for ICMP

    Horizon allows for ICMP to be type:code.
    Type and code can be from -1 to 255.

    API refers to both EC2 and Nova APIs

    This patch attempts to resolve:
    1. API code throws exceptations when 0 is passed for either field
    2. API code validates type:code like from->to range. type and code
       are independent
    3. Update unit tests for this new set of operations.

    A side effect is that the following are allowed type:code.
    -1:X
    X:-1

    The code assumes that -1 is a wildcard for the field.

    bug 956967

    Change-Id: Ieb6989815afc6986b72e0efc7611c2cc353ab5d8