diff options
author | truckman <truckman@FreeBSD.org> | 2017-04-27 07:30:48 +0000 |
---|---|---|
committer | truckman <truckman@FreeBSD.org> | 2017-04-27 07:30:48 +0000 |
commit | b5cebfa8d51591353618c9e05272f84af512540d (patch) | |
tree | 1882bad572cc32a6ccc5033985d3347607a5e130 | |
parent | 2dc97f4906e1348af64f0e50db125824bc443c52 (diff) | |
download | FreeBSD-src-b5cebfa8d51591353618c9e05272f84af512540d.zip FreeBSD-src-b5cebfa8d51591353618c9e05272f84af512540d.tar.gz |
MFC r316777 (by cem)
dummynet: Use strlcpy to appease static checkers
Some dummynet modules used strcpy() to copy from a larger buffer
(dn_aqm->name) to a smaller buffer (dn_extra_parms->name). It happens that
the lengths of the strings in the dn_aqm buffers were always hardcoded to be
smaller than the dn_extra_parms buffer ("CODEL", "PIE").
Use strlcpy() instead, to appease static checkers. No functional change.
Reported by: Coverity
CIDs: 1356163, 1356165
Sponsored by: Dell EMC Isilon
-rw-r--r-- | sys/netpfil/ipfw/dn_aqm_codel.c | 2 | ||||
-rw-r--r-- | sys/netpfil/ipfw/dn_aqm_pie.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/sys/netpfil/ipfw/dn_aqm_codel.c b/sys/netpfil/ipfw/dn_aqm_codel.c index 0080170..a095b33 100644 --- a/sys/netpfil/ipfw/dn_aqm_codel.c +++ b/sys/netpfil/ipfw/dn_aqm_codel.c @@ -416,7 +416,7 @@ aqm_codel_getconfig(struct dn_fsk *fs, struct dn_extra_parms * ep) struct dn_aqm_codel_parms *ccfg; if (fs->aqmcfg) { - strcpy(ep->name, codel_desc.name); + strlcpy(ep->name, codel_desc.name, sizeof(ep->name)); ccfg = fs->aqmcfg; ep->par[0] = ccfg->target / AQM_TIME_1US; ep->par[1] = ccfg->interval / AQM_TIME_1US; diff --git a/sys/netpfil/ipfw/dn_aqm_pie.c b/sys/netpfil/ipfw/dn_aqm_pie.c index c2e4d43..85062e2 100644 --- a/sys/netpfil/ipfw/dn_aqm_pie.c +++ b/sys/netpfil/ipfw/dn_aqm_pie.c @@ -755,7 +755,7 @@ aqm_pie_getconfig (struct dn_fsk *fs, struct dn_extra_parms * ep) { struct dn_aqm_pie_parms *pcfg; if (fs->aqmcfg) { - strcpy(ep->name, pie_desc.name); + strlcpy(ep->name, pie_desc.name, sizeof(ep->name)); pcfg = fs->aqmcfg; ep->par[0] = pcfg->qdelay_ref / AQM_TIME_1US; ep->par[1] = pcfg->tupdate / AQM_TIME_1US; |