c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2025-01-03 23:09:35 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix two minor lint errors (cppcheck)

Browse Source 
Fix two minor errors reported by cppcheck:

In module/zfs/abd.c (abd_get_offset_impl), add non-NULL
assertion to prevent NULL dereference warning.

In module/zfs/arc.c (l2arc_write_buffers), change 'try'
variable to 'pass' to avoid C++ reserved word.

Reviewed-by: Ryan Moeller <ryan@iXsystems.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Colm Buckley <colm@tuatha.org>
Closes #11507
This commit is contained in:
Colm 2021-01-23 23:49:32 +00:00 committed by Brian Behlendorf
parent 12fec4a147
commit 0f3b928e85
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
module/zfs/abd.c
View File

@ -557,6 +557,7 @@ abd_get_offset_impl(abd_t *sabd, size_t off, size_t size)
abd = abd_get_offset_scatter(sabd, off);
}
ASSERT3P(abd, !=, NULL);
abd->abd_size = size;
abd->abd_parent = sabd;
zfs_refcount_create(&abd->abd_children);

8
module/zfs/arc.c
View File

@ -8937,17 +8937,17 @@ l2arc_write_buffers(spa_t *spa, l2arc_dev_t *dev, uint64_t target_sz)
/*
* Copy buffers for L2ARC writing.
*/
for (int try = 0; try < L2ARC_FEED_TYPES; try++) {
for (int pass = 0; pass < L2ARC_FEED_TYPES; pass++) {
/*
* If try == 1 or 3, we cache MRU metadata and data
* If pass == 1 or 3, we cache MRU metadata and data
* respectively.
*/
if (l2arc_mfuonly) {
if (try == 1 || try == 3)
if (pass == 1 || pass == 3)
continue;
}
multilist_sublist_t *mls = l2arc_sublist_lock(try);
multilist_sublist_t *mls = l2arc_sublist_lock(pass);
uint64_t passed_sz = 0;
VERIFY3P(mls, !=, NULL);