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 linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <1470217425-19709-1-git-send-email-phil@nwl.cc> Date: Wed, 3 Aug 2016 11:43:45 +0200 From: Phil Sutter <phil@....cc> To: Stephen Hemminger <shemming@...cade.com> Cc: netdev@...r.kernel.org Subject: [iproute PATCH] tc: Fix for missing estimator initialization When switching to C99 initializers, I forgot to add this one. This means that when trying to set an estimator value, tc would complain about spurious duplicate estimator parameter. But much worse, the random variable content is sent to the kernel regardless of whether an estimator was given or not. Fixes: d17b136f7d7dd ("Use C99 style initializers everywhere") Reported-by: Stas Nichiporovich <stasn77@...il.com> Signed-off-by: Phil Sutter <phil@....cc> --- tc/tc_qdisc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tc/tc_qdisc.c b/tc/tc_qdisc.c index bc87aab110e48..93c9a4c1daa28 100644 --- a/tc/tc_qdisc.c +++ b/tc/tc_qdisc.c @@ -45,7 +45,7 @@ static int usage(void) static int tc_qdisc_modify(int cmd, unsigned int flags, int argc, char **argv) { struct qdisc_util *q = NULL; - struct tc_estimator est; + struct tc_estimator est = {}; struct { struct tc_sizespec szopts; __u16 *data; -- 2.8.2
Powered by blists - more mailing lists