Author: lstewart
Date: Mon Mar 11 08:21:43 2013
New Revision: 248150
URL: http://svnweb.freebsd.org/changeset/base/248150
Log:
MFC r247906:
The hashmask returned by hashinit() is a valid index in the returned
hash array. Fix a siftr(4) potential memory leak and INVARIANTS
triggered kernel panic in hashdestroy() by ensuring the last array index
in the flow counter hash table is flushed of entries.
Approved by: re (kib)
Modified:
stable/8/sys/netinet/siftr.c
Directory Properties:
stable/8/sys/ (props changed)
stable/8/sys/netinet/ (props changed)
Modified: stable/8/sys/netinet/siftr.c
==============================================================================
--- stable/8/sys/netinet/siftr.c Mon Mar 11 07:55:49 2013
(r248149)
+++ stable/8/sys/netinet/siftr.c Mon Mar 11 08:21:43 2013
(r248150)
@@ -1319,7 +1319,7 @@ siftr_manage_ops(uint8_t action)
* flow seen and freeing any malloc'd memory.
* The hash consists of an array of LISTs (man 3 queue).
*/
- for (i = 0; i < siftr_hashmask; i++) {
+ for (i = 0; i <= siftr_hashmask; i++) {
LIST_FOREACH_SAFE(counter, counter_hash + i, nodes,
tmp_counter) {
key = counter->key;
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
