diff options
| author | luozhengzheng <[email protected]> | 2016-12-09 05:48:09 +0800 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2016-12-08 14:48:09 -0700 |
| commit | c077090a9b11d42c0734f0741ca08d0c9b6b37b0 (patch) | |
| tree | 4a26fe274cff85f60167f8fd803f312f6e1ca883 | |
| parent | f95e64789157c2fa373cb424f2b5713be4c8a639 (diff) | |
Fix coverity defects: CID 154617
CID 154617: Memory - illegal accesses (UNINIT)
The value here just needs to be initialized to make Coverity happy.
When dsize == 0, then value of daiter.iter_mapaddr is irrelevant. That
address won't be accessed, it's only used for some arithmetic. dsize
can be zero either if dabd is null, or if code column is longer than the
current data column.
Reviewed-by: Gvozden Neskovic <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: luozhengzheng <[email protected]>
Closes #5437
| -rw-r--r-- | module/zfs/abd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/module/zfs/abd.c b/module/zfs/abd.c index d14564b33..15d998a3b 100644 --- a/module/zfs/abd.c +++ b/module/zfs/abd.c @@ -1321,7 +1321,7 @@ abd_raidz_gen_iterate(abd_t **cabds, abd_t *dabd, int i; ssize_t len, dlen; struct abd_iter caiters[3]; - struct abd_iter daiter; + struct abd_iter daiter = {0}; void *caddrs[3]; unsigned long flags; |