Bug 1182107 - assertion failure with very high MTU
Summary: assertion failure with very high MTU
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: openvswitch
Version: 22
Hardware: All
OS: Linux
high
high
Target Milestone: ---
Assignee: Flavio Leitner
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-01-14 13:01 UTC by Flavio Leitner
Modified: 2015-10-29 23:26 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1182191 1182193 (view as bug list)
Environment:
Last Closed: 2015-10-29 23:26:40 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Flavio Leitner 2015-01-14 13:01:07 UTC
Update OVS package to include the following two fixes:

commit 3282e5118707efc3ba07f7bd17f5f82fe931a732                                
Author: Ben Pfaff <blp>
Date:   Sun Jan 11 13:45:36 2015 -0800

    dpif-linux: Drop oversized packets instead of assert-failing.
    
    A packet sent to a Netlink datapath has to fit within a Netlink attribute.
    Until now, this was only checked in an assertion inside the Netlink code,
    which meant that trying to send a too-large packet (approximate 64 kB or
    larger) would assert-fail.  It's better to just drop those packets, which
    this commit does.
    
    Reported-by: Shuping Cui <scui>
    Reported-by: Jiri Benc <jbenc>
    Signed-off-by: Ben Pfaff <blp>
    Acked-by: Jesse Gross <jesse>

commit f3c7ec6a2a19bffdfa5218f984bda53582ecb8dc
Author: Ben Pfaff <blp>
Date:   Wed Jan 7 13:19:41 2015 -0800

    netlink: Refine calculation of maximum-length attributes.
    
    Until now the Netlink code has considered an attribute to exceed the
    maximum length if the *padded* size of the attribute exceeds 65535 bytes.
    For example, an attribute with a 65529-byte payload, together with 4-byte
    header and 3 bytes of padding, takes up 65536 bytes and therefore the
    existing code rejected it.
    
    However, the restriction on Netlink attribute sizes is to ensure that the
    length fits in the 16-bit nla_len field.  This field includes the 4-byte
    header but not the padding, so a 65529-byte payload is acceptable because,
    with the header but not the padding, it comes to only 65533 bytes.
    
    Thus, this commit relaxes the restriction on Netlink attribute sizes by
    omitting padding from size checks.  It also changes one piece of code that
    inlined a size check to use the central function nl_attr_oversized().
    
    This change should fix an assertion failure when OVS userspace passes a
    maximum-size (65529+ byte) packet back to the kernel.
    
    Reported-by: Shuping Cui <scui>
    Reported-by: Jiri Benc <jbenc>
    Signed-off-by: Ben Pfaff <blp>
    Acked-by: Jesse Gross <jesse>

Comment 1 Jaroslav Reznik 2015-03-03 17:06:54 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 22 development cycle.
Changing version to '22'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora22


Note You need to log in before you can comment on or make changes to this bug.