jobd: fix accidental scalarification in ferror()
authorJan Krüger <jk@jk.gs>
Wed, 24 Nov 2010 19:28:01 +0000 (24 20:28 +0100)
committerKyle J. McKay <mackyle@gmail.com>
Fri, 31 May 2013 11:37:01 +0000 (31 04:37 -0700)
Turns out that sprintf() expects a scalar as its first argument, so we
unwittingly turned the entire argument list of ferror() into a scalar.
So, instead, use the first argument separately when calling sprintf().

Signed-off-by: Jan Krüger <jk@jk.gs>
jobd/jobd.pl

index 9b8b748..0c78057 100755 (executable)
@@ -336,7 +336,7 @@ sub error($) {
        print STDERR ts().shift()."\n";
 }
 sub ferror(@) {
-       error(sprintf(@_));
+       error(sprintf($_[0], @_[1..$#_]));
 }
 sub fatal($) {
        error(shift);