Browse Source

Revert the following commit as it breaks hostname.if(5) lines with a

backslash at the end for line continuation
Breaking long lines into multiple ones must still be possible and does
require to treat the backslash as an escape character.
Breakage reported by Mark Patruck <mark at wrapped dot cx >, thanks!
---
distrib/miniroot/install.sub revision 1.1151
etc/netstart revision 1.203
date: 2020/05/21 11:54:41;  author: kn;  state: Exp;  lines: +2 -2;
Do not treat backslashe as an escape character in hostname.if(5) lines
ifstart() should always pass such lines unaltered, especially if they
contain "nwid" or "description" lines with arbitrary strings.
<bsdlisten at gmail dot com> reported SSIDs such as "Mike's" during
installation end as broken;  this was because the installer escaped
the single quote using backslashes which ended up being treated as
escape characters much later during hostname.if parsing in netstart(8).
Ok deraadt
master
kn 4 years ago
parent
commit
62f534bc48
1 changed files with 2 additions and 2 deletions
  1. +2
    -2
      src/etc/netstart

+ 2
- 2
src/etc/netstart View File

@ -1,6 +1,6 @@
#!/bin/sh - #!/bin/sh -
# #
# $OpenBSD: netstart,v 1.204 2020/05/21 13:42:02 kn Exp $
# $OpenBSD: netstart,v 1.205 2020/05/22 13:38:44 kn Exp $
# Turn off Strict Bourne shell mode. # Turn off Strict Bourne shell mode.
set +o sh set +o sh
@ -133,7 +133,7 @@ ifstart() {
# Parse the hostname.if(5) file and fill _cmds array with interface # Parse the hostname.if(5) file and fill _cmds array with interface
# configuration commands. # configuration commands.
set -o noglob set -o noglob
while IFS= read -r -- _line; do
while IFS= read -- _line; do
parse_hn_line $_line parse_hn_line $_line
done <$_hn done <$_hn


Loading…
Cancel
Save