From a63a002c533be0b8aa502379aaf689c5a41ec6b2 Mon Sep 17 00:00:00 2001
From: Darren 'Tadgy' Austin <darren@afterdark.org.uk>
Date: Sat, 20 Jul 2019 21:48:06 +0100
Subject: [PATCH] Use '' around declared values, and handle escaping in
 variable expansion.

---
 parse_ini | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/parse_ini b/parse_ini
index c4616fb..c85d27c 100755
--- a/parse_ini
+++ b/parse_ini
@@ -357,8 +357,6 @@ parse_ini() {
         # Strip the []s and any whitespace between the []s and the section name.
         LINE="${LINE/#\[*([[:space:]])/}"
         LINE="${LINE/%*([[:space:]])\]/}"
-#        LINE="${LINE/#*([[:blank:]])\[*([[:blank:]])/}"
-#        LINE="${LINE/%*([[:blank:]])\]*([[:blank:]])/}"
 
         # Squash multiple consecutive blanks into a single space.
         ((SQUASH_SPACES == 1)) && LINE="${LINE//+([[:blank:]])/ }"
@@ -474,9 +472,9 @@ parse_ini() {
 
       # Output the associative array element definition.
       if ((DUPLICATES_MERGE == 0)); then
-        ((CHECK_ONLY == 0)) && printf "%s%s%s[\"%s\"]=\"%s\"\\n" "$PREFIX" "${PREFIX:+$DELIM}" "$CURRENT_SECTION" "$KEY" "$VALUE"
+        ((CHECK_ONLY == 0)) && printf "%s%s%s[\"%s\"]='%s'\\n" "$PREFIX" "${PREFIX:+$DELIM}" "$CURRENT_SECTION" "$KEY" "${VALUE//\'/\'\\\'\'}"
       else
-        ((CHECK_ONLY == 0)) && printf "%s%s%s[\"%s\"]+=\"%s\"\\n" "$PREFIX" "${PREFIX:+$DELIM}" "$CURRENT_SECTION" "$KEY" "$VALUE"
+        ((CHECK_ONLY == 0)) && printf "%s%s%s[\"%s\"]+='%s'\\n" "$PREFIX" "${PREFIX:+$DELIM}" "$CURRENT_SECTION" "$KEY" "${VALUE//\'/\'\\\'\'}"
       fi
     else
       ((CHECK_ONLY == 1)) && echo "${0##*/}: line $LINENUMBER: skipping line" >&2