Return -EINVAL from _from_string() functions

We'd return -1 (-EPERM), even though we have a general rule to use real errno
values. The particular case that caught my attention was:

$ sudo udevadm control -l asdf
Failed to parse log priority 'asdf': Operation not permitted

... but "git grep 'r =.*_from_string' src/" return 110 hits. Confusingly, some
of the _from_string functions already return a proper errno value, so not all
of those are broken, but probably quite a few.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/shared/test-tables.h b/src/shared/test-tables.h
index e68ec60497..90210b9b59 100644
--- a/src/shared/test-tables.h
+++ b/src/shared/test-tables.h
@@ -33,11 +33,11 @@ static inline void _test_table(const char *name,
 
                 if (i >= 0 && i < size) {
                         if (sparse)
-                                assert_se(rev == i || rev == -1);
+                                assert_se(rev == i || rev == -EINVAL);
                         else
                                 assert_se(val != NULL && rev == i);
                 } else
-                        assert_se(val == NULL && rev == -1);
+                        assert_se(val == NULL && rev == -EINVAL);
         }
 }