internal/report: fix issue with lint/fix in unexclude

Some operations in "unexclude" were incorrectly operating
on the old report instead of the new report; fix this.

Change-Id: Iaae67a6292bb10e0840e9841cccd0ffe1cc5d4ba
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/576995
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Damien Neil <dneil@google.com>

Author: tatianab

cmd/vulnreport/unexclude.go

@@ -89,10 +89,8 @@ func (u *unexclude) run(ctx context.Context, filename string) (err error) {
 	// Remove description because this is a "basic" report.
 	newR.Description = ""
 
-	r.Fix(u.pc)
-	if hasLints := newR.LintAsNotes(u.pc); hasLints {
-		log.Warnf("unexcluded report %s has lint errors that need to be fixed manually", filename)
-	}
+	newR.Fix(u.pc)
+	hasLints := newR.LintAsNotes(u.pc)
 
 	if err := os.Remove(filename); err != nil {
 		log.Errf("could not remove excluded report: %v", err)
@@ -103,6 +101,11 @@ func (u *unexclude) run(ctx context.Context, filename string) (err error) {
 	if err != nil {
 		return err
 	}
+
+	if hasLints {
+		log.Warnf("unexcluded report %s has lint errors that need to be fixed manually", newFilename)
+	}
+
 	log.Out(newFilename)
 
 	return nil