Skip to content

Commit

Permalink
No need to return error and updated comments
Browse files Browse the repository at this point in the history
Co-authored-by: Leonardo Grasso <[email protected]>
Signed-off-by: GLVSKiriti <[email protected]>
  • Loading branch information
GLVSKiriti and leogr committed Mar 26, 2024
1 parent af195e1 commit 7a390e9
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions events/syscall/remove_bulk_data_from_disk.go
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,9 @@ func RemoveBulkDataFromDisk(h events.Helper) error {
}

h.Log().Infof("attempting to run shred command to remove bulk data from disk")
// Rule triggers regardless of whether the 'shred' utility exists
// Therefore, there's no need to skip the action even if 'shred' does not exist
// Rule triggers regardless of whether the 'shred' utility exists or its outcome
// Therefore, there's no need to skip the action or report the error in any case
cmd := exec.Command("shred", "-u", filename)
err := cmd.Run()
return err
_ := cmd.Run()

Check failure on line 38 in events/syscall/remove_bulk_data_from_disk.go

View workflow job for this annotation

GitHub Actions / build-and-test-amd64

no new variables on left side of :=

Check failure on line 38 in events/syscall/remove_bulk_data_from_disk.go

View workflow job for this annotation

GitHub Actions / build-and-test-arm64

no new variables on left side of :=
return nil
}

0 comments on commit 7a390e9

Please sign in to comment.