diff options
| author | Richard Yao <[email protected]> | 2014-04-23 00:04:41 -0400 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2014-05-30 17:02:37 -0700 |
| commit | 2024041b6c5134a925a33c10eff24a47ecb541a6 (patch) | |
| tree | cab29b947802495f0a37b0dc965a9b6da65b4449 /cmd/zstreamdump/zstreamdump.c | |
| parent | 4def05f8a68f7255dfb88606a9b475314c828a06 (diff) | |
Remove superfluous statement
Clang's static analyzer reported that the value assigned to pcksum is
never used. That is because we initialize both zc and pcksum to {{ 0 }}
and then do `pcksum = zc;`. That is fairly pointless. However, it has
the effect of generating a false positive in Clang's static analyzer.
Since noise from false positives can obscure real issues, we fix it
anyway.
Signed-off-by: Richard Yao <[email protected]>
Signed-off-by: Ned Bass <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Issue #2330
Diffstat (limited to 'cmd/zstreamdump/zstreamdump.c')
| -rw-r--r-- | cmd/zstreamdump/zstreamdump.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/cmd/zstreamdump/zstreamdump.c b/cmd/zstreamdump/zstreamdump.c index c5bcdabd3..a4c451d53 100644 --- a/cmd/zstreamdump/zstreamdump.c +++ b/cmd/zstreamdump/zstreamdump.c @@ -127,7 +127,6 @@ main(int argc, char *argv[]) } send_stream = stdin; - pcksum = zc; while (ssread(drr, sizeof (dmu_replay_record_t), &zc)) { if (first) { |