8dc3f4842b
Merge pull request #7493 from keszybz/revert-revert
...
Revert "meson: drop rootprefix option"
2017-11-29 18:56:26 +01:00
a7af7362a3
Merge pull request #7504 from keszybz/two-mkosi-ideas
...
Two mkosi tweaks
2017-11-29 16:24:14 +01:00
abc861e865
Merge pull request #7500 from poettering/journal-field-count
...
fix journald field counts
2017-11-29 16:21:48 +01:00
22ecadbee2
Add mkosi.output/ to .gitignore
...
It's not something that would ever be checked in, and mkosi along
with systemd, so this directory is likely to appear.
2017-11-29 14:33:56 +01:00
c7e501475a
mkosi: do not build man pages
2017-11-29 14:33:55 +01:00
9a8e64b0e9
meson: emit a warning if rootprefix is set
...
It's printed at the end because it's the easiest to spot there.
When meson is upgraded to 0.43.1, we'll be able to use warning() instead.
2017-11-29 13:29:59 +01:00
e61ad5c283
journald: correct field counts
...
N_IOVEC_OBJECT_FIELDS is bumped 14 → 18 (see dispatch_message_real() and
count!)
N_IOVEC_PAYLOAD_FIELDS is bumped 15 → 16 (see
server_space_usage_message() and count!)
Also, add comments, to make clear what is what.
2017-11-29 11:37:21 +01:00
f643ae7171
journal: driver messages can now contain object fields, account for that
...
In some cases we can now log about processes, hence we must keep room
for that.
2017-11-29 11:36:22 +01:00
3235b4e70c
Merge pull request #7492 from keszybz/coverity-fixes
...
A few more cleanups based on coverity and valgrind
2017-11-29 12:50:49 +09:00
ae34a10bac
udev/collect: use _cleanup_ in one more place
...
This doesn't make the code pretty, but it at least makes it a tiny bit shorter.
2017-11-28 21:34:50 +01:00
3176084ff2
udev/collect: remove dead conditional
...
Coverity rightly says that 'word' is assigned to 'ptr', and 'word'
was assigned from 'ptr' (yikes) right after a non-null check.
2017-11-28 21:34:50 +01:00
2504834861
journal: avoid undefined behaviour in float division by 0.0
...
Coverity says that's undefined. I'm pretty sure we always would get a nan, but
let's avoid (formally) undefined behaviour since that can cause compilers to do
strange things.
2017-11-28 21:34:50 +01:00
b921fcb2ac
networkd: fix memleak of Rules
...
This was the issue mentioned in 93f9da6e0a
2017-11-28 21:34:50 +01:00
f916819053
journal: use new helpers with journal_file_close
...
journal_file_close_set() is not necessary anymore.
2017-11-28 21:34:50 +01:00
1599f593d6
journal-remote: add helper function and use hashmap_free_with_destructor
2017-11-28 21:34:46 +01:00
6dd1c36839
cryptsetup-generator: add helper function and use hashmap_free_with_destructor
2017-11-28 21:34:41 +01:00
39ab16e397
coredump: fix typo and use hashmap_free_with_destructor
2017-11-28 21:34:34 +01:00
e85b096b43
machine-image: convert image_hashmap_free() to an static inline helper
2017-11-28 21:30:43 +01:00
224b0e7ad0
Add set/hashmap helpers for non-trivial freeing and use where straighforward
...
A macro is needed because otherwise we couldn't ensure type safety.
Some simple tests are included.
No functional change intended.
2017-11-28 21:30:30 +01:00
ec1d290903
Use hashmap_free_free where appropriate
2017-11-28 21:26:37 +01:00
74344a1702
Revert "meson: drop rootprefix option"
...
This reverts commit ee4bfa21f6#7461 .
2017-11-28 20:02:53 +01:00
5c4912ea61
Merge pull request #7490 from yuwata/test-basic
...
add more tests for basic functions
2017-11-28 18:15:30 +01:00
fbdb6605b5
networkd: Add comment for setting_mtu #7460 ( #7489 )
...
Follow up #7460
2017-11-28 17:24:52 +01:00
4de8d3f59e
Merge pull request #7486 from keszybz/coverity-fixes
...
Fixes for issues spotted by coverity (and some cleanups)
2017-11-28 15:04:11 +01:00
23cc81e7c2
test-cap-list: add more tests about capability_set_{from,to}_string()
2017-11-28 22:17:00 +09:00
4c1a95fd84
test: move tests about cap_list_cap() from test-cap-list to test-capability
2017-11-28 22:14:54 +09:00
db4bd5bd62
cap-list: check range of numeric value
2017-11-28 22:06:34 +09:00
6b86b9e65d
test-bitmap: add tests for bitmap_copy()
2017-11-28 20:34:36 +09:00
ade0eb2190
test-architecture: add more tests
2017-11-28 20:34:32 +09:00
f30788ee23
test-alloc-util: add tests for memdup_suffix0() and greedy_realloc()
2017-11-28 20:33:58 +09:00
a4e6e06bc7
tests: add tests for empty string
2017-11-28 20:21:25 +09:00
d40c54fe56
alloc-util: coding style fix
2017-11-28 20:21:15 +09:00
b39367a1a7
Merge pull request #7398 from keszybz/udev-list
...
udev: use list.h instead of udev_list_node
2017-11-28 11:35:02 +01:00
c9f034a10a
Merge pull request #7484 from poettering/sd-resolve-timeout
...
timeout sd-resolve tests
2017-11-28 11:33:38 +01:00
ba77dac291
pid1: improve syntax in error message
2017-11-28 09:26:14 +01:00
d387cfdddd
pid1: clarify that min_max is always initialized
...
read_one_line_file() always returns <= 0, so the code was OK, but let's write
the check a bit differently to make it obvious that min_max is always set.
2017-11-28 09:26:14 +01:00
7beb3baf16
test-hostname-util: add assert_se's to make coverity happy
2017-11-28 09:26:14 +01:00
c47a9f909a
test-process-util: add (void) cast to make coverity happy
...
The intent is for the call to succeed only when privileged, so make
that clear.
2017-11-28 09:25:39 +01:00
ca9b2b4dc1
test-fd-util: add assert_se to make coverity happy
2017-11-28 09:25:39 +01:00
b0fe73d740
udevadm-monitor: add some assert_se's to make coverity happy
...
We do that elsewhere and it's nice to be consistent.
2017-11-28 09:25:38 +01:00
595c66a3db
core/umount: use _cleanup_ in one more place
2017-11-28 09:25:38 +01:00
1a96c8e1cc
sd-resolve: check that name fits in the specified packet length
...
Coverity complained that we didn't check if the data is long enough.
2017-11-28 09:25:38 +01:00
8d44f5a637
shim: fix logically dead code
...
The alternate verification path could never be taken.
Found by coverity.
2017-11-28 09:25:38 +01:00
1e40b7490a
networkd: remove dead code
...
Found by coverity.
2017-11-28 09:25:38 +01:00
ef51180b62
test-ndisc-ra: add assert_se() to make coverity happy
2017-11-28 09:25:38 +01:00
10a6e01eaa
boot/shim: fix char ** vs char * mismatch
...
This code cannot have ever worked ;(
Found by coverity, obvious when the type is not obfuscated to void*.
2017-11-28 09:25:38 +01:00
bceda88b52
boot/efi: fix NULL dereference
...
The comment above makes the intent of the code pretty clear:
"use security2_protocol == NULL as indicator".
So revert the condition in the check and fix the logic in the comment while
at it.
The question is how this could have ever worked: if BS->LocateProtocol
(which is supposedly optional) ever failed, we'd crash here. Strange.
Found by coverity.
2017-11-28 09:25:38 +01:00
bdd73ac5c2
dissect-image: simplify check for passphrase presence
...
m->encrypted is set when fstype=="crypto_LUKS", but this is not obvious when
reading decrypt_partition(). Just check if passphrase is set before using
it.
2017-11-28 09:25:38 +01:00
bbd1590085
networkd: fix reversed arguments
...
Found by coverity.
Also drop unused return value while at it.
2017-11-28 09:25:38 +01:00
93f9da6e0a
networkd: plug some memleaks in rule serialization/deserialization
...
This fixes the (mostly theoretical, since we're only parsing data that we write
ourselves) memleak when iif or oif is deserialized multiple times. Unfortunately
it does not fix the memleak when rule is freed, but that'll require a bigger
effort.
2017-11-28 09:25:38 +01:00
Lennart Poettering