about summary refs log tree commit homepage
path: root/lib/PublicInbox/SolverGit.pm
diff options
context:
space:
mode:
authorEric Wong <e@yhbt.net>2020-01-11 22:34:58 +0000
committerEric Wong <e@yhbt.net>2020-01-13 04:36:40 +0000
commit587cc7c14629a59fd39c9c5b991f0676217722e0 (patch)
treebc8da833bffc9cef64165b3dcf35f73a4fec4052 /lib/PublicInbox/SolverGit.pm
parented3e1ad3bb9c07835dee97c76b4454b98faf0dcd (diff)
downloadpublic-inbox-587cc7c14629a59fd39c9c5b991f0676217722e0.tar.gz
This avoids uninitialized variable warnings when viewing
newly-created files.
Diffstat (limited to 'lib/PublicInbox/SolverGit.pm')
-rw-r--r--lib/PublicInbox/SolverGit.pm4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/PublicInbox/SolverGit.pm b/lib/PublicInbox/SolverGit.pm
index b48e8ac4..baad981d 100644
--- a/lib/PublicInbox/SolverGit.pm
+++ b/lib/PublicInbox/SolverGit.pm
@@ -169,7 +169,7 @@ sub extract_diff ($$) {
         my $patch = $9;
 
         # don't care for leading 'a/' and 'b/'
-        my (undef, @a) = split(m{/}, git_unquote($path_a));
+        my (undef, @a) = split(m{/}, git_unquote($path_a)) if defined($path_a);
         my (undef, @b) = split(m{/}, git_unquote($path_b));
 
         # get rid of path-traversal attempts and junk patches:
@@ -177,7 +177,7 @@ sub extract_diff ($$) {
         state $bad_component = { map { $_ => 1 } ('', '.', '..') };
         foreach (@a, @b) { return if $bad_component->{$_} }
 
-        $di->{path_a} = join('/', @a);
+        $di->{path_a} = join('/', @a) if @a;
         $di->{path_b} = join('/', @b);
 
         my $path = ++$self->{tot};