fix: extract retry logic into function for BusyBox ash compatibility

BusyBox ash does not support the 'local' keyword outside of
functions. The previous implementation used 'local pending_status'
directly in the main loop, which caused ash to silently crash.

Move the retry logic into a retry_pending_report() function
defined before the main loop, where 'local' is valid.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Codex

openwisp-config/files/openwisp.agent

@@ -987,6 +987,19 @@ else
 	env -i ACTION="restart" /sbin/hotplug-call openwisp
 fi
 
+retry_pending_report() {
+	local pending_status
+	pending_status=$(cat "$PENDING_REPORT")
+	logger "Retrying pending report_status: $pending_status" \
+		-t openwisp \
+		-p daemon.info
+	retry_with_backoff report_status "$pending_status"
+	# shellcheck disable=SC2181
+	if [ "$?" -eq "0" ]; then
+		rm -f "$PENDING_REPORT"
+	fi
+}
+
 while true; do
 	# check management interface at each iteration
 	# (because the address may change due to
@@ -1001,15 +1014,7 @@ while true; do
 		# Retry a previously failed report_status without re-downloading.
 		# This handles the case where the config was applied successfully
 		# but report_status failed due to a transient server error.
-		local pending_status
-		pending_status=$(cat "$PENDING_REPORT")
-		logger "Retrying pending report_status: $pending_status" \
-			-t openwisp \
-			-p daemon.info
-		retry_with_backoff report_status "$pending_status"
-		if [ "$?" -eq "0" ]; then
-			rm -f "$PENDING_REPORT"
-		fi
+		retry_pending_report
 	fi
 	env -i ACTION="end-of-cycle" /sbin/hotplug-call openwisp