refactor: [minor refactor] remove extra line and return nil to not br…

…eak existing mount functionalities

Signed-off-by: Shivanjan Chakravorty <[email protected]>

pkg/chattr/chattr.go

@@ -16,14 +16,13 @@ const (
 	lsattrCmd = "lsattr"
 )
 
-func AddImmutable(path string) (err error) {
-
+func AddImmutable(path string) error {
 	return ForceAddImmutable(path, false)
 }
 
-func ForceAddImmutable(path string, force bool) (err error) {
+func ForceAddImmutable(path string, force bool) error {
 	chattrBin := which(chattrCmd)
-	if _, err = os.Stat(path); err == nil {
+	if _, err := os.Stat(path); err == nil {
 		var cmd *exec.Cmd
 		if force {
 			cmd = exec.Command(chattrBin, "+i", "-f", path)
@@ -46,16 +45,15 @@ func ForceAddImmutable(path string, force bool) (err error) {
 			}
 			return fmt.Errorf("%s +i failed: %s. Err: %v", chattrBin, stderr.String(), err)
 		}
-		return nil
 	}
 
-	return err
+	return nil
 }
 
-func RemoveImmutable(path string) (err error) {
+func RemoveImmutable(path string) error {
 	return ForceRemoveImmutable(path, false)
 }
-func ForceRemoveImmutable(path string, force bool) (err error) {
+func ForceRemoveImmutable(path string, force bool) error {
 	chattrBin := which(chattrCmd)
 	if _, err := os.Stat(path); err == nil {
 		var cmd *exec.Cmd
@@ -80,10 +78,9 @@ func ForceRemoveImmutable(path string, force bool) (err error) {
 			}
 			return fmt.Errorf("%s -i failed: %s. Err: %v", chattrBin, stderr.String(), err)
 		}
-		return nil
 	}
 
-	return err
+	return nil
 }
 
 func IsImmutable(path string) bool {