From 06d931fab61ac49ee64646982e2c5600ed4c619c Mon Sep 17 00:00:00 2001
From: Michael Tremer <michael.tremer@ipfire.org>
Date: Fri, 29 Mar 2019 12:22:39 +0100
Subject: [PATCH] Add message to get consent from user

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
---
 src/firmware-update.in | 39 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 38 insertions(+), 1 deletion(-)

diff --git a/src/firmware-update.in b/src/firmware-update.in
index 6e1f75a..08c74b2 100644
--- a/src/firmware-update.in
+++ b/src/firmware-update.in
@@ -26,6 +26,39 @@ PACKAGE_VERSION="@PACKAGE_VERSION@"
 
 FIRMWARE_DIR="@firmwaredir@"
 
+get_consent() {
+	local firmware="${1}"
+
+	cat <<EOF
+
+You are going to flash the firmware on device. This is a dangerous operation
+that might potentially damage your device.
+
+Please do NOT TURN OFF the system while the update is in progress and do
+not abort the process.
+
+  Firmware file: ${firmware}
+
+EOF
+
+	local answer
+	while [ -t 0 -a -t 1 -a -t 2 ]; do
+		# Wait for user to type y
+		printf "Do you want to proceed? [y/n] "
+		read -r answer
+
+		case "${answer}" in
+			y|Y)
+				return 0
+				;;
+			n|N)
+				echo "Aborting."
+				return 1
+				;;
+		esac
+	done
+}
+
 read_dmi() {
 	local what="${1}"
 
@@ -141,8 +174,12 @@ update_pcengines_apu() {
 		return 1
 	fi
 
+	echo "New firmware version available: ${new_version}"
+
 	# Perform update
-	do_flashrom -p "internal" -w "${firmware}"
+	if get_consent "${firmware}"; then
+		do_flashrom -p "internal" -w "${firmware}"
+	fi
 }
 
 do_update() {
-- 
2.47.3