diff options
| author | Phil Sutter <phil@nwl.cc> | 2016-08-03 11:43:45 +0200 |
|---|---|---|
| committer | Stephen Hemminger <shemming@brocade.com> | 2016-08-06 10:14:06 -0700 |
| commit | 9579afb24e9c56c6fc4c89e2e99fe4b3de304200 (patch) | |
| tree | 5de374df431e3275729258dc188f419f6826cb03 /tc/tc_qdisc.c | |
| parent | e3d0f0c0e3d8ac6432884e19aefd169e5c4ae179 (diff) | |
| download | platform_external_iproute2-9579afb24e9c56c6fc4c89e2e99fe4b3de304200.tar.gz platform_external_iproute2-9579afb24e9c56c6fc4c89e2e99fe4b3de304200.tar.bz2 platform_external_iproute2-9579afb24e9c56c6fc4c89e2e99fe4b3de304200.zip | |
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@gmail.com>
Signed-off-by: Phil Sutter <phil@nwl.cc>
Diffstat (limited to 'tc/tc_qdisc.c')
| -rw-r--r-- | tc/tc_qdisc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tc/tc_qdisc.c b/tc/tc_qdisc.c index bc87aab1..93c9a4c1 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; |