Hard crash entering `ctrl-^` then `.` then `enter` into `blink>` prompt about blink HOT 7 CLOSED

That's an interesting one. Will take a look at what may be provoking the crash.

from blink.

I thought people won't type weird things and closed #1900. I was wrong.

Edit: #1900 is not actually relevant, because this is valid UTF-8. The correspondence comes from "weird things typed".

from blink.

Well I did not really input it intentionally, it just happened ;-). Still would be good if that didn't cause a hard crash

from blink.

No worries. I didn't mean to be harsh about this. I hold the same opinion that a crash or freeze should not happen.

This crash happens inside ios_system and I can reproduce it in a-shell too. I'm having a hard time getting Address Sanitizer to work inside ios_system, but there is some progress.

from blink.

Passing the string 1E 00 (Ctrl-^) to getLastCharacterOfArgument() returns an invalid pointer 0x1. This needs to be fixed upstream.

P.S. Address Sanitizer was not needed for this.

from blink.

@carloscabanero I have a patch for it. It's very unfortunate that ios_system uses ^^ as its internal Record Separator for arguments with both ' and ". The internal ^^ always comes in pairs, except when you put one in the input ;)

diff --git a/ios_system.m b/ios_system.m
index 21e9eeb..87a498b 100644
--- a/ios_system.m
+++ b/ios_system.m
@@ -2459,6 +2459,7 @@ static char* getLastCharacterOfArgument(const char* argument) {
         return NULL;
     } else if (argument[0] == recordSeparator) {
         char* endquote = strchr(argument + 1, recordSeparator);
+        if (endquote == NULL) return NULL; // be safe
         return endquote + 1;
     }
     // TODO: the last character of the argument could also be '<' or '>' (vim does that, with no space after file name)

from blink.

Done 17.3.0 . Thanks for the help

from blink.