diff options
| author | Eric Wong <e@80x24.org> | 2020-11-10 03:20:29 +0000 |
|---|---|---|
| committer | Eric Wong <e@80x24.org> | 2020-11-10 18:15:12 +0000 |
| commit | 0246654927fd9ddc9718ca0d7d5c7e29ce004d5c (patch) | |
| tree | 1bdc805aa9961b8be71c83e70dd0f30cc477af00 /lib/PublicInbox | |
| parent | df8e4a0612545d53672036641e9f076efc94c2f6 (diff) | |
| download | public-inbox-0246654927fd9ddc9718ca0d7d5c7e29ce004d5c.tar.gz | |
In case of other bugs or intentional corruption of over.sqlite3, we don't want to attempt dereferencing a non-ref scalar when calling ->mid_delete in the fallback code path. Noticed while chasing another bug in extindex development...
Diffstat (limited to 'lib/PublicInbox')
| -rw-r--r-- | lib/PublicInbox/SearchIdx.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/PublicInbox/SearchIdx.pm b/lib/PublicInbox/SearchIdx.pm index 06d1a9f5..662055c6 100644 --- a/lib/PublicInbox/SearchIdx.pm +++ b/lib/PublicInbox/SearchIdx.pm @@ -554,8 +554,8 @@ sub unindex_eml { $tmp{$_}++ for @removed; } if (!$nr) { - $mids = join('> <', @$mids); - warn "W: <$mids> missing for removal from overview\n"; + my $m = join('> <', @$mids); + warn "W: <$m> missing for removal from overview\n"; } while (my ($num, $nr) = each %tmp) { warn "BUG: $num appears >1 times ($nr) for $oid\n" if $nr != 1; |