daemon: fix uninitialized variable

And also replace an unnecessary substitution (s///) op with a
match (m//).

Fixes: 93a7b219d58aad86 ("public-inbox-netd: a multi-protocol superserver")

diff --git a/lib/PublicInbox/Daemon.pm b/lib/PublicInbox/Daemon.pm
index 13cce1ec51abcf3499363e388b359db6bd7f4a1a..d08ce0f9a4148905e05b5b8321afeb9a12552350 100644 (file)
--- a/lib/PublicInbox/Daemon.pm
+++ b/lib/PublicInbox/Daemon.pm
@@ -618,9 +618,9 @@ sub daemon_loop ($) {
        while (my ($k, $v) = each %tls_opt) {
                my $l = $k;
                $l =~ s!\A([^:]+)://!!;
-               my $scheme = $1;
+               my $scheme = $1 // '';
                my $xn = $xnetd->{$l} // $xnetd->{''};
-               if ($scheme =~ s!\A(?:https|imaps|nntps)!!) {
+               if ($scheme =~ m!\A(?:https|imaps|nntps)!) {
                        $post_accept{$l} = tls_start_cb($v, $xn->{post_accept});
                } elsif ($xn->{tlsd}) { # STARTTLS, $k eq '' is OK
                        $xn->{tlsd}->{accept_tls} = $v;