Added proper null/empty checks and variable quoting in 3 files:
1. wordpress-cron-manager.sh (2 issues):
- Added validation for $site_path before use
- Quoted variable in cron command to prevent word splitting
- Lines 446-449: Check if path is empty or invalid before processing
2. malware-scanner.sh (1 issue):
- Added safety check for $SCAN_DIR before suggesting rm -rf command
- Prevents dangerous rm operations if variable is empty or root
- Line 1583-1585: Guard against accidental deletions
3. mysql-restore-to-sql.sh (2 issues):
- Quoted $datadir in echo statements showing manual commands
- Lines 426, 441, 444, 447: Proper quoting in examples
Impact: Prevents potential issues from empty/undefined variables
Automatically detects when missing tablespace errors are unrelated to the
selected database and recommends Force Recovery Level 1.
Changes:
- Added selected_database parameter to show_recovery_options()
- Detects if missing files are from selected DB vs other DBs
- Shows clear recommendation when missing files are ONLY from other databases
- Explains that Force Recovery Level 1 is safe and correct for selective restore
- Prevents user confusion when restoring single DB from full backup
Use case:
When user restores ibdata1 + single database (e.g., amea_wp) from a full backup,
ibdata1 contains metadata for all databases. Script now detects this and says:
'SMART DETECTION: Missing files are from OTHER databases, not amea_wp'
'Your selected database amea_wp appears to have all files!'
'RECOMMENDED ACTION: Use Force Recovery Level 1'
This eliminates confusion and guides users to the correct solution.
The intelligent recovery system wasn't detecting missing .ibd files because
MariaDB/MySQL error format uses 'was not found at' instead of 'missing'.
Changes:
- Added 'was not found at' pattern to grep searches (3 locations)
- Enhanced tablespace extraction to parse './db/table.ibd' format
- Extracts database/table from error: 'Tablespace N was not found at ./db/table.ibd'
- Falls back to quoted tablespace name extraction if new pattern doesn't match
Now when script detects missing .ibd files it will:
- Show DIAGNOSIS: Missing or unopenable tablespace files
- List exact missing tables with database names
- Provide copy-paste ready cp commands
- Show all recovery options instead of generic troubleshooting
- Removed control panel path documentation from script header
(system-detect.sh already documents and shows this when it runs)
- Changed detect_control_panel from silent (>/dev/null) to visible output
so users see what control panel was detected and which paths will be used
- Added comment explaining SYS_USER_HOME_BASE usage
Added comprehensive documentation to script header:
- Lists all 4 control panel paths (cPanel, Plesk, InterWorx, standalone)
- References source: lib/system-detect.sh -> SYS_USER_HOME_BASE
- Documents InterWorx special case (/chroot/home vs /home symlink)
- Shows restore directory and SQL output directory formats
- Makes it clear where paths come from for maintenance
Changes to modules/backup/mysql-restore-to-sql.sh:
Multi-Control Panel Support:
- Source system-detect.sh to detect control panel
- Use SYS_USER_HOME_BASE for restore directory paths
- cPanel/InterWorx/Standalone: /home
- Plesk: /var/www/vhosts
- Fixes issue where InterWorx/Plesk don't have /home directories
SQL Output Location Fix:
- Changed output from current working directory to restore directory
- SQL files now saved to parent of TEMP_DATADIR
Example: /home/temp/restore20251210/ (not /root/)
- Prevents cluttering control panel system directories
- Added print_info showing exact save location before dump
Safety Enhancements:
- Added check_disk_space() function (validates 2x required space)
- Added warn_force_recovery() function (levels 5-6 require risk acknowledgment)
- Integrated disk space check before dump creation
- Integrated force recovery warnings in step4_configure_options()
- Added cleanup trap handler for Ctrl+C/interruption
- Critical safety check prevents using /var/lib/mysql as restore dir
Changes to REFDB_FORMAT.txt:
- Documented multi-control panel support
- Added control_panel_paths section with all 4 panel paths
- Updated output location documentation
- Added safety features documentation
- Updated features list
QA Status: ✅ PASSED
- 0 CRITICAL issues
- 0 HIGH issues
- Syntax validated
- All safety checks functional
cschantz