lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Sun, 7 Apr 2019 10:29:48 +0800
From:   linmiaohe <linmiaohe@...wei.com>
To:     David Miller <davem@...emloft.net>, <dsa@...ulusnetworks.com>
CC:     <shrijeet@...il.com>, <netdev@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <mingfangsen@...wei.com>
Subject: Re: [PATCH net] net: vrf: Fix ping failed when vrf mtu is set to 0



On 2019/4/7 9:12, David Miller wrote:
> From: linmiaohe <linmiaohe@...wei.com>
> Date: Thu, 4 Apr 2019 20:01:13 +0800
> 
>> From: Miaohe Lin <linmiaohe@...wei.com>
>>
>> When the mtu of a vrf device is set to 0, it would cause ping
>> failed.So I think we should limit dev->min_mtu to ETH_MIN_MTU
>> to solve this problem. And if dev->max_mtu still be 0 can be
>> confusing, so I limit dev->max_mtu to ETH_MAX_MTU.
>>
>> Here is the reproduce step:
> 
> Just in case it isn't clear, David Ahern gave you feedback and
> asked you to adjust things so that the problem is solved for ipv6
> as well.  He also gave you an appropriate Fixes: tag to add to
> your commit message.
> 
> .
> 

I'am sorry, I'am on my holiday, so I found these emalis just now.
I will adjust things in v2 so that this patch will solve for ipv6 as well.
And add an appropriate fixes to my commit message.
Thank you very much.

Powered by blists - more mailing lists