net/dns: fix blank lines being added to resolv.conf on OpenBSD (#13928)

During resolv.conf update, old 'search' lines are cleared but '\n' is not
deleted, leaving behind a new blank line on every update.

This adds 's' flag to regexp, so '\n' is included in the match and deleted when
old lines are cleared.

Also, insert missing `\n` when updated 'search' line is appended to resolv.conf.

Signed-off-by: Renato Aguiar <renato@renatoaguiar.net>
pull/13936/head
Renato Aguiar 1 day ago committed by GitHub
parent 9d1348fe21
commit 5d07c17b93
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

@ -57,6 +57,7 @@ func (m *resolvdManager) SetDNS(config OSConfig) error {
if len(newSearch) > 1 { if len(newSearch) > 1 {
newResolvConf = append(newResolvConf, []byte(strings.Join(newSearch, " "))...) newResolvConf = append(newResolvConf, []byte(strings.Join(newSearch, " "))...)
newResolvConf = append(newResolvConf, '\n')
} }
err = m.fs.WriteFile(resolvConf, newResolvConf, 0644) err = m.fs.WriteFile(resolvConf, newResolvConf, 0644)
@ -123,6 +124,6 @@ func (m resolvdManager) readResolvConf() (config OSConfig, err error) {
} }
func removeSearchLines(orig []byte) []byte { func removeSearchLines(orig []byte) []byte {
re := regexp.MustCompile(`(?m)^search\s+.+$`) re := regexp.MustCompile(`(?ms)^search\s+.+$`)
return re.ReplaceAll(orig, []byte("")) return re.ReplaceAll(orig, []byte(""))
} }

Loading…
Cancel
Save