Bug 203923

Summary: makefs: Coverity CID 975621: False positive
Product: Base System Reporter: scdbackup
Component: binAssignee: Ed Maste <emaste>
Status: Closed Not A Bug    
Severity: Affects Only Me CC: emaste
Priority: ---    
Version: CURRENT   
Hardware: Any   
OS: Any   

Description scdbackup 2015-10-21 12:43:20 UTC
usr.sbin/makefs/cd9660/cd9660_write.c

CID 975621 (#1 of 1): Copy-paste error (COPY_PASTE_ERROR)
   copy_paste_error: rf in fclose(rf) looks like a copy-paste error.
   Should it say fd instead?

471                        (void)fclose(rf);

--------------- Source analysis:

The suspicion stems from the assymetric reaction on error.
Only the read FILE pointer rf was opened inside the function.
So it gets closed in both error cases: with read FILE *rf and
with write FILE *fd.

          if (ferror(rf)) {
                  ...
                  (void)fclose(rf);
          ...
          if (ferror(fd)) {
                  ...
                  (void)fclose(rf);

--------------- Remedy proposal:

In Coverity classify CID 975621 as "False positive" and set its Action
to "Ignore".
Comment 1 Enji Cooper freebsd_committer freebsd_triage 2015-10-25 22:13:00 UTC
Bulk taking makefs bugs.
Comment 2 Enji Cooper freebsd_committer freebsd_triage 2017-11-05 20:47:23 UTC
Releasing bugs back to the pool.
Comment 3 Ed Maste freebsd_committer freebsd_triage 2023-03-23 17:27:28 UTC
Have marked this as a false positive in Coverity.