diff options
author | Gregory Greenman <gregory.greenman@intel.com> | 2017-11-26 15:51:31 +0200 |
---|---|---|
committer | Luca Coelho <luciano.coelho@intel.com> | 2017-12-05 21:01:42 +0200 |
commit | f6f046f0c0b74bb45ba990a632033f77c1202a82 (patch) | |
tree | c2deb13013c6e577168ddb12419f5b3431ee3fae /drivers/net/wireless/intel/iwlwifi/mvm/debugfs.c | |
parent | 177a11cf1c4d671fad701b7cf12bbe9ad7c28781 (diff) |
iwlwifi: mvm: rs: add size checks when printing to a buffer
Replace sprintf by scnprintf throughout rs code.
Signed-off-by: Gregory Greenman <gregory.greenman@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
Diffstat (limited to 'drivers/net/wireless/intel/iwlwifi/mvm/debugfs.c')
-rw-r--r-- | drivers/net/wireless/intel/iwlwifi/mvm/debugfs.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/debugfs.c b/drivers/net/wireless/intel/iwlwifi/mvm/debugfs.c index 366fa75f9091..834c798f728c 100644 --- a/drivers/net/wireless/intel/iwlwifi/mvm/debugfs.c +++ b/drivers/net/wireless/intel/iwlwifi/mvm/debugfs.c @@ -460,7 +460,8 @@ static ssize_t iwl_dbgfs_rs_data_read(struct file *file, char __user *user_buf, "last tx rate=0x%X ", lq_sta->last_rate_n_flags); - desc += rs_pretty_print_rate(buff + desc, lq_sta->last_rate_n_flags); + desc += rs_pretty_print_rate(buff + desc, bufsz - desc, + lq_sta->last_rate_n_flags); mutex_unlock(&mvm->mutex); ret = simple_read_from_buffer(user_buf, count, ppos, buff, desc); @@ -973,7 +974,8 @@ static ssize_t iwl_dbgfs_frame_stats_read(struct iwl_mvm *mvm, continue; pos += scnprintf(pos, endpos - pos, "Rate[%d]: ", (int)(ARRAY_SIZE(stats->last_rates) - i)); - pos += rs_pretty_print_rate(pos, stats->last_rates[idx]); + pos += rs_pretty_print_rate(pos, endpos - pos, + stats->last_rates[idx]); } spin_unlock_bh(&mvm->drv_stats_lock); |