Skip to content

Commit

Permalink
Correct conditional logic in PIF check
Browse files Browse the repository at this point in the history
The original conditional expression used incorrect syntax for grouping within [[ ]]. This commit corrects the logic to use || (OR) instead of && (AND) and removes the erroneous parentheses. This ensures the script accurately checks if the Play Integrity Fix module directory exists and contains a valid module.prop file.
  • Loading branch information
0x11DFE committed Nov 21, 2024
1 parent 33a75a8 commit b6db2f5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion magisk_module_files/action.sh
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ PlayIntegrityFix() {
PIF_DIRS="$PIF_MODULE_DIR/pif.json"

# Check if PIF_MODULE_DIR is a directory and module.prop exists and is not empty
if [[ ! (-d "$PIF_MODULE_DIR" && -s "$PIF_MODULE_DIR/module.prop") ]]; then
if [[ ! -d "$PIF_MODULE_DIR" || ! -s "$PIF_MODULE_DIR/module.prop" ]]; then
abort "PlayIntegrityFix module is missing or is not accessible, Skipping !"
fi

Expand Down

0 comments on commit b6db2f5

Please sign in to comment.