[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <230a5b4f-2cbf-4911-5231-b05bf6a44571@gmail.com>
Date: Tue, 7 Dec 2021 20:01:16 -0700
From: David Ahern <dsahern@...il.com>
To: "lizhijian@...itsu.com" <lizhijian@...itsu.com>,
"davem@...emloft.net" <davem@...emloft.net>,
"kuba@...nel.org" <kuba@...nel.org>,
"shuah@...nel.org" <shuah@...nel.org>
Cc: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"linux-kselftest@...r.kernel.org" <linux-kselftest@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Jie2x Zhou <jie2x.zhou@...el.com>,
"Li Zhijian(intel)" <zhijianx.li@...el.com>
Subject: Re: [PATCH v2] selftests: net: Correct case name
On 12/6/21 11:05 PM, lizhijian@...itsu.com wrote:
>> #################################################################
>> With VRF
>>
>> TEST: Raw socket bind to local address - ns-A IP [ OK ]
>> TEST: Raw socket bind to local address after device bind - ns-A IP [ OK ]
>> TEST: Raw socket bind to local address after VRF bind - ns-A IP [ OK ]
>> TEST: Raw socket bind to local address - VRF IP [FAIL]
>>
>
> i found that above case failed with "server: error binding socket: 99: Cannot assign requested address"
> i have manually check it with below command after setup(), same errors:
>
> # ip netns exec ns-A nettest -s -R -P icmp -l 172.16.3.1 -b
> 05:55:11 server: error binding socket: 99: Cannot assign requested address
>
> But when i specified specific network interface, it works
> # ip netns exec ns-A nettest -s -R -P icmp -l 172.16.3.1 -b -I red
> # echo $?
> 0
> # ip netns exec ns-A nettest -s -R -P icmp -l 172.16.3.1 -b
> 06:01:55 server: error binding socket: 99: Cannot assign requested address
> # echo $?
> 1
>
>
> So i wonder if i missed something ?
>
That test should be a negative test as is the first one in that group -
in both cases the address bind should fail since the socket is not in
the VRF but the address is. The first on currently shows "OK" but that
is because of 5cad8bce26e01 that made changes to the config to validate
MD5 changes. Will send a patch to fix.
Powered by blists - more mailing lists