diff options
author | Lin Ma <[email protected]> | 2023-07-25 10:42:27 +0800 |
---|---|---|
committer | Jakub Kicinski <[email protected]> | 2023-07-26 22:08:14 -0700 |
commit | 6c58c8816abb7b93b21fa3b1d0c1726402e5e568 (patch) | |
tree | 85750ad130476c432682926965b10b9b7e4738db /lib/test_fortify/write_overflow-strncpy-src.c | |
parent | 0f0fa27b871b53a62c3cd2b054ebcce199277310 (diff) |
net/sched: mqprio: Add length check for TCA_MQPRIO_{MAX/MIN}_RATE64
The nla_for_each_nested parsing in function mqprio_parse_nlattr() does
not check the length of the nested attribute. This can lead to an
out-of-attribute read and allow a malformed nlattr (e.g., length 0) to
be viewed as 8 byte integer and passed to priv->max_rate/min_rate.
This patch adds the check based on nla_len() when check the nla_type(),
which ensures that the length of these two attribute must equals
sizeof(u64).
Fixes: 4e8b86c06269 ("mqprio: Introduce new hardware offload mode and shaper in mqprio")
Reviewed-by: Victor Nogueira <[email protected]>
Signed-off-by: Lin Ma <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'lib/test_fortify/write_overflow-strncpy-src.c')
0 files changed, 0 insertions, 0 deletions