Do not leak file descriptors in backup_crontab

Originally, if anything went wrong during the backup,
the early return caused the crontab_file and possibly backup_file
pointers to leak.

Issue found by static scanner.

src/crontab.c

@@ -594,18 +594,19 @@ static int backup_crontab(const char *crontab_path) {
 	}
 
 	if (retval != 0)
-		return retval;
+		goto cleanup;
 
 	if (EOF != ch)
 		while (EOF != (ch = get_char(crontab_file)))
 			putc(ch, backup_file);
 
-	(void) fclose(crontab_file);
-	(void) fclose(backup_file);
-
 	printf("Backup of %s's previous crontab saved to %s\n", User, backup_path);
 
-	return 0;
+cleanup:
+	if (backup_file) (void) fclose(backup_file);
+	(void) fclose(crontab_file);
+
+	return retval;
 }
 
 static void check_error(const char *msg) {