[backend] catch errors in the publish filter

We do not want to crash the scheduler, but return a meaningful error.
openSUSE · May 27, 2024 · 100ae72 · 100ae72
1 parent 75fe30b
commit 100ae72
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/backend/BSSched/PublishRepo.pm b/src/backend/BSSched/PublishRepo.pm
@@ -183,7 +183,12 @@ sub prpfinished {
   $filter = $bconf->{'publishfilter'} if $bconf;
   undef $filter if $filter && !@$filter;
   $filter ||= $default_publishfilter;
-  $filter = compile_publishfilter($filter);
+  eval { $filter = compile_publishfilter($filter) };
+  if ($@) {
+    my $err = $@;
+    chomp $err;
+    return "invalid publish filter: $err";
+  }
 
   my $seen_binary;
   my $singleexport;