Message ID | 1626056297-5744-1-git-send-email-xuyang2018.jy@fujitsu.com |
---|---|
State | Changes Requested |
Headers | show |
Series | [v3] network/tc01.sh: Add a regression test for tc qdisc command | expand |
Hi Xu! On 12.07.2021 05:18, Yang Xu wrote: > Signed-off-by: Yang Xu <xuyang2018.jy@fujitsu.com> > --- > runtest/net.tcp_cmds | 1 + > testcases/network/tcp_cmds/tc/Makefile | 13 +++++++ > testcases/network/tcp_cmds/tc/tc01.sh | 47 ++++++++++++++++++++++++++ > 3 files changed, 61 insertions(+) > create mode 100644 testcases/network/tcp_cmds/tc/Makefile > create mode 100755 testcases/network/tcp_cmds/tc/tc01.sh > > diff --git a/runtest/net.tcp_cmds b/runtest/net.tcp_cmds > index db47dfd5b..7e142de11 100644 > --- a/runtest/net.tcp_cmds > +++ b/runtest/net.tcp_cmds > @@ -12,6 +12,7 @@ netstat netstat01.sh > ping01 ping01.sh > ping02 ping02.sh > sendfile sendfile01.sh > +tc01 tc01.sh > tcpdump tcpdump01.sh > telnet telnet01.sh > iptables iptables01.sh > diff --git a/testcases/network/tcp_cmds/tc/Makefile b/testcases/network/tcp_cmds/tc/Makefile > new file mode 100644 > index 000000000..60150a1ce > --- /dev/null > +++ b/testcases/network/tcp_cmds/tc/Makefile > @@ -0,0 +1,13 @@ > +# SPDX-License-Identifier: GPL-2.0-or-later > +# Copyright (C) 2009, Cisco Systems Inc. > +# Ngie Cooper, July 2009 Since it's a new Makefile, not sure if it should be copy-pasted from the other tests... > + > +top_srcdir ?= ../../../.. > + > +include $(top_srcdir)/include/mk/env_pre.mk > + > +INSTALL_TARGETS := tc01.sh > + > +MAKE_TARGETS := There is no point adding an empty make targets here. > + > +include $(top_srcdir)/include/mk/generic_leaf_target.mk > diff --git a/testcases/network/tcp_cmds/tc/tc01.sh b/testcases/network/tcp_cmds/tc/tc01.sh > new file mode 100755 > index 000000000..a9da45b41 > --- /dev/null > +++ b/testcases/network/tcp_cmds/tc/tc01.sh > @@ -0,0 +1,47 @@ > +#!/bin/sh > +# SPDX-License-Identifier: GPL-2.0-or-later > +# Copyright (c) 2021 FUJITSU LIMITED. All rights reserved. > +# Author: Yang Xu<xuyang2018.jy@fujitsu.com> > +# > +# When using "tc qdisc add dev teql0 root teql0 command", qdisc_create() > +# calls teql_qdisc_init() it imediately fails after check "if (m->dev == dev)" > +# because both devices are teql0, and it does not set qdisc_priv(sch)->m > +# leaving it zero on error path, then qdisc_create() imediately calls > +# teql_destroy() which does not expect zero master pointer and we get OOPS > +# on unpatched kernel. > +# > +# If we enable panic_on_oops, this case may crash. > +# > +# This kernel bug was introduced by > +# commit 87b60cfacf9f ("net_sched: fix error recovery at qdisc creation") > +# and has been fixed by > +# commit 1ffbc7ea9160 ("net: sched: sch_teql: fix null-pointer dereference") > +# > + > +TST_SETUP="setup" > +TST_TESTFUNC="do_test" > +TST_NEEDS_ROOT=1 > +TST_NEEDS_DRIVERS="sch_teql" > +TST_NEEDS_CMDS="tc modprobe dmesg grep" > + > +. tst_test.sh > + > +setup() > +{ > + ROD modprobe $TST_NEEDS_DRIVERS > +} > + > +do_test() > +{ > + tst_res TINFO "Use tc qdisc command to trigger a null-pointer dereference" > + > + EXPECT_FAIL tc qdisc add dev teql0 root teql0 > + > + if dmesg | grep -q 'RIP:.*sch_teql'; then > + tst_res TFAIL "This bug is reproduced." > + else > + tst_res TPASS "This bug is not reproduced." > + fi > +} > + > +tst_run > For the rest: Reviewed-by: Alexey Kodanev <aleksei.kodanev@bell-sw.com>
diff --git a/runtest/net.tcp_cmds b/runtest/net.tcp_cmds index db47dfd5b..7e142de11 100644 --- a/runtest/net.tcp_cmds +++ b/runtest/net.tcp_cmds @@ -12,6 +12,7 @@ netstat netstat01.sh ping01 ping01.sh ping02 ping02.sh sendfile sendfile01.sh +tc01 tc01.sh tcpdump tcpdump01.sh telnet telnet01.sh iptables iptables01.sh diff --git a/testcases/network/tcp_cmds/tc/Makefile b/testcases/network/tcp_cmds/tc/Makefile new file mode 100644 index 000000000..60150a1ce --- /dev/null +++ b/testcases/network/tcp_cmds/tc/Makefile @@ -0,0 +1,13 @@ +# SPDX-License-Identifier: GPL-2.0-or-later +# Copyright (C) 2009, Cisco Systems Inc. +# Ngie Cooper, July 2009 + +top_srcdir ?= ../../../.. + +include $(top_srcdir)/include/mk/env_pre.mk + +INSTALL_TARGETS := tc01.sh + +MAKE_TARGETS := + +include $(top_srcdir)/include/mk/generic_leaf_target.mk diff --git a/testcases/network/tcp_cmds/tc/tc01.sh b/testcases/network/tcp_cmds/tc/tc01.sh new file mode 100755 index 000000000..a9da45b41 --- /dev/null +++ b/testcases/network/tcp_cmds/tc/tc01.sh @@ -0,0 +1,47 @@ +#!/bin/sh +# SPDX-License-Identifier: GPL-2.0-or-later +# Copyright (c) 2021 FUJITSU LIMITED. All rights reserved. +# Author: Yang Xu<xuyang2018.jy@fujitsu.com> +# +# When using "tc qdisc add dev teql0 root teql0 command", qdisc_create() +# calls teql_qdisc_init() it imediately fails after check "if (m->dev == dev)" +# because both devices are teql0, and it does not set qdisc_priv(sch)->m +# leaving it zero on error path, then qdisc_create() imediately calls +# teql_destroy() which does not expect zero master pointer and we get OOPS +# on unpatched kernel. +# +# If we enable panic_on_oops, this case may crash. +# +# This kernel bug was introduced by +# commit 87b60cfacf9f ("net_sched: fix error recovery at qdisc creation") +# and has been fixed by +# commit 1ffbc7ea9160 ("net: sched: sch_teql: fix null-pointer dereference") +# + +TST_SETUP="setup" +TST_TESTFUNC="do_test" +TST_NEEDS_ROOT=1 +TST_NEEDS_DRIVERS="sch_teql" +TST_NEEDS_CMDS="tc modprobe dmesg grep" + +. tst_test.sh + +setup() +{ + ROD modprobe $TST_NEEDS_DRIVERS +} + +do_test() +{ + tst_res TINFO "Use tc qdisc command to trigger a null-pointer dereference" + + EXPECT_FAIL tc qdisc add dev teql0 root teql0 + + if dmesg | grep -q 'RIP:.*sch_teql'; then + tst_res TFAIL "This bug is reproduced." + else + tst_res TPASS "This bug is not reproduced." + fi +} + +tst_run
Signed-off-by: Yang Xu <xuyang2018.jy@fujitsu.com> --- runtest/net.tcp_cmds | 1 + testcases/network/tcp_cmds/tc/Makefile | 13 +++++++ testcases/network/tcp_cmds/tc/tc01.sh | 47 ++++++++++++++++++++++++++ 3 files changed, 61 insertions(+) create mode 100644 testcases/network/tcp_cmds/tc/Makefile create mode 100755 testcases/network/tcp_cmds/tc/tc01.sh