summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* build-sys: suppress warnings of unused resultsFilipe Brandenburger2015-06-101-0/+1
| | | | | | | | | | | | | | | | | | | | Unfortunately, gcc keeps warning about those even when we use an explicit (void) cast to indicate we are not interested in the result. LLVM's clang does not have that issue and works fine with the casts. GCC bug being tracked at: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66425 Until that GCC bug is fixed (and the version with the fix is in many/most distributions) or we switch to LLVM as the default compiler, it looks like we'll have to disable this warning by default... Tested by building files known to present warnings about unused results without the suppression, confirmed that the warnings were no longer present with this patch applied. This partially reverts commit 00c11bc53a1d37 ("build-sys: don't suppress irrelevant warnings").
* Merge pull request #3 from threatgrid/more_cgtop_enhancementsLennart Poettering2015-06-112-26/+50
|\ | | | | cgtop enhancements for easier machine-readable output
| * cgtop: IO readings are valid if any data is available, even if unchanged ↵Charles Duffy2015-06-101-2/+1
| | | | | | | | | | | | | | since last tick Emit "0" rather than "-" if no change in IO values are seen for a process since last tick, so long as accounting has registered content at all.
| * cgtop: more sensible flushing behavior w/ non-TTY outputCharles Duffy2015-06-091-2/+8
| | | | | | | | | | | | - Explicitly flush stdout before sleep between iterations - Only clear user keystrokes when output is to TTY - Add a newline between output batches when output is not to TTY
| * cgtop: allow user to force looping behavior even in non-TTY modeCharles Duffy2015-06-092-9/+11
| |
| * cgtop: raw output option (disable conversion to human-readable units)Charles Duffy2015-06-092-13/+30
| |
* | Merge pull request #6 from xnox/drop-nameLennart Poettering2015-06-114-26/+22
|\ \ | | | | | | shared: Drop 'name=' prefix from SYSTEMD_CGROUP_CONTROLLER define.
| * | shared: Drop 'name=' prefix from SYSTEMD_CGROUP_CONTROLLER define.Dimitri John Ledkov2015-06-024-26/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In cgtop,mount-setup,nspawn the name= prefix is hard-coded in the mount options, and the define is not used. Everywhere else, we explicitly white-list allow 'name=' prefix to be used with all controllers, and strip it out to 'normalise' the controller name. That work is mostly inflicted on us due to 'name=' prefix in the define. Dropping this prefix makes everything more sane overall.
* | | Merge pull request #152 from zonque/buildsysLennart Poettering2015-06-101-2/+2
|\ \ \ | | | | | | | | build-sys: make man/systemd.directives.xml depend on man/custom-entit…
| * | | build-sys: make man/systemd.directives.xml depend on man/custom-entities.entDaniel Mack2015-06-101-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the following command sequence fails: make distclean ./autogen.sh c make distcheck That's because the command invoked to build man/systemd.directives.xml needs man/custom-entities.ent to function, which itself isn't a dependency. The $(filter-out $<,$^) logic used to filter out everything from the prerequisites except for the first word, which doesn't work anymore now. Use $(SOURCE_XML_FILES) instead.
* | | | Merge pull request #148 from teg/sd-network-raceLennart Poettering2015-06-101-7/+56
|\ \ \ \ | | | | | | | | | | sd-network: allow the state dir to be created after the monitor
| * | | | sd-network: allow the state dir to be created after the monitorTom Gundersen2015-06-101-7/+56
| | | | | | | | | | | | | | | | | | | | | | | | | We now listen for new subdirs of /run/systemd, and /run/systemd/netif in case /run/systemd/netif/links does not exist.
* | | | | Merge pull request #151 from mischief/dns-shared-3Lennart Poettering2015-06-1014-38/+61
|\ \ \ \ \ | |_|/ / / |/| | | | move dns code from resolve to shared v3
| * | | | libsystemd-network: use domain validation instead of hostname validation for ↵Nick Owens2015-06-102-6/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dhcp domain option previously hostname_is_valid was used to validate domain names, which would silently drop perfectly valid dns names that were longer than a single dns label.
| * | | | shared: add convenience function for validating dns namesNick Owens2015-06-101-0/+9
| | | | |
| * | | | resolve: move dns routines into sharedNick Owens2015-06-1012-32/+30
| | | | |
* | | | | Include tools/compile-unifont.py in the source tarballMike Gilbert2015-06-101-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | This prevents a build failure when /usr/share/fonts/unifont/unifont.hex is newer than unifont-glyph-array.bin.
* | | | | bus: we now support path_namespace=/David Herrmann2015-06-102-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | Our bloom-filters support root-path matching. Make sure we properly add the path_namespace= tag.
* | | | | bus: fix pattern matchingDavid Herrmann2015-06-103-8/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DBus-spec defines two different pattern matchings: 1) Path and namespace prefix matching. In this case, A matches B either if both are equal, or if B is fully included in the namespace of A. In other words, A has to be a prefix of B, but end with a separator character (or the following character in B must be one). This is used for path_namespace= and arg0namespace= 2) The other pattern matching is used for arg0path= which does a two-way matching. That is, A must be a prefix of B, or B a prefix of A. Furthermore, the prefix must end with a separator. Fix the sd-bus helpers to reflect that. The 'simple_' and 'complex_' prefixes don't make any sense now, but.. eh..
* | | | | bus: fix test-bus-kerne-bloom.c to match properlyDavid Herrmann2015-06-101-3/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure we actually verify our match-rules are executed properly. Right now all we test is the bloom-matches, which are non-reliable as they leave through false-positives.
* | | | | bus: fix arg0path= two-way matchingDavid Herrmann2015-06-101-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DBus spec clearly defines arg0path= to be a two-way matching. That is, either the matcher or the matchee can be a prefix of the other to match. This is not possible to implement with bloom-filters. Instead, we'd have to add a separate filter for each prefix. This is non-trivial, though. Hence, just skip the match for now and match locally.
* | | | | bus: fix bloom_add_prefixes() to add all required dataDavid Herrmann2015-06-101-1/+9
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Lets look at an example where we add arg0="/foo/bar/waldo" to a bloom-filter. The following strings are added: "arg0:/foo/bar/waldo" "arg0-slash-prefix:/foo/bar" "arg0-slash-prefix:/foo" Two problems arise: 1) If we match on "arg0path=/foo/bar/waldo", the dbus-spec explicitly states that equal strings are also considered prefixes. However, in the bloom-match, we can only provide a single match-filter. Therefore, we have to add "arg0-slash-prefix:/foo/bar/waldo" there, but this never occured in the bloom-mask of the message. Hence, this patch makes sure bloom_add_prefixes() adds the full path as prefix, too. 2) If we match on "arg0path=/foo/", the dbus-spec states that arg0path does prefix-matching with the trailing slash _included_, unlike path_namespace= matches, which does *not* include them. This is inconsistent, but we have to support the specs. Therefore, we must add prefixes with _and_ without trailing separators. Hence, this patch makes sure bloom_add_prefixes() adds all prefixes with the trailing slash included. The final set of strings added therefore is: "arg0:/foo/bar/waldo" "arg0-slash-prefix:/foo/bar/waldo" "arg0-slash-prefix:/foo/bar/" "arg0-slash-prefix:/foo/bar" "arg0-slash-prefix:/foo/" "arg0-slash-prefix:/foo" "arg0-slash-prefix:/"
* | | | Merge pull request #147 from poettering/cmsgDaniel Mack2015-06-1015-21/+28
|\ \ \ \ | | | | | | | | | | util: introduce CMSG_FOREACH() macro and make use of it everywhere
| * | | | util: introduce CMSG_FOREACH() macro and make use of it everywhereLennart Poettering2015-06-1015-21/+28
| | | | | | | | | | | | | | | | | | | | | | | | | It's only marginally shorter then the usual for() loop, but certainly more readable.
* | | | | Merge pull request #146 from arvidjaar/pr/udev-import-program-exit-code-manTom Gundersen2015-06-101-1/+2
|\ \ \ \ \ | |_|/ / / |/| | | | man: clarify that IMPORT{program} is done only for zero exit code
| * | | | man: clarify that IMPORT{program} is done only for zero exit codeAndrei Borzenkov2015-06-101-1/+2
| |/ / /
* | | | Fix typoZbigniew Jędrzejewski-Szmek2015-06-101-1/+1
|/ / / | | | | | | | | | Follow up for 7c918141ed.
* | | Merge pull request #142 from teg/sd-network-unref-NULLLennart Poettering2015-06-101-4/+4
|\ \ \ | | | | | | | | sd-network: allow NULL in sd_network_monitor_unref
| * | | sd-network: allow NULL in sd_network_monitor_unrefTom Gundersen2015-06-101-4/+4
| | | | | | | | | | | | | | | | Match rest of codebase, we always allow unref'ing NULL.
* | | | Merge pull request #140 from teg/man-udev-timeoutDaniel Mack2015-06-101-1/+1
|\ \ \ \ | | | | | | | | | | man: udevd - correct default event timeout
| * | | | man: udevd - correct default event timeoutTom Gundersen2015-06-101-1/+1
| |/ / / | | | | | | | | | | | | This was changed from 30 to 180 seconds quite some time ago.
* | | | Merge pull request #85 from keszybz/selinux-contextZbigniew Jędrzejewski-Szmek2015-06-107-103/+40
|\ \ \ \
| * | | | journald: simplify context handlingZbigniew Jędrzejewski-Szmek2015-06-101-24/+8
| | | | | | | | | | | | | | | | | | | | By using our homegrown function we can dispense with all the iffdefery.
| * | | | sd-bus: store selinux context at connection timeZbigniew Jędrzejewski-Szmek2015-06-104-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This appears to be the right time to do it for SOCK_STREAM unix sockets. Also: condition bus_get_owner_creds_dbus1 was reversed. Split it out to a separate variable for clarity and fix. https://bugzilla.redhat.com/show_bug.cgi?id=1224211
| * | | | sd-bus: do not use per-datagram auxiliary informationZbigniew Jędrzejewski-Szmek2015-06-101-68/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SELinux information cannot be retrieved this way, since we are using stream unix sockets and SCM_SECURITY does not work for them. SCM_CREDENTIALS use dropped to be consistent. We also should get this information at connection time. https://bugzilla.redhat.com/show_bug.cgi?id=1224211 "SCM_SECURITY was only added for datagram sockets."
| * | | | bus-creds: always set SD_BUS_CREDS_PID when we set pid in the maskZbigniew Jędrzejewski-Szmek2015-06-092-8/+7
| | | | | | | | | | | | | | | | | | | | Also reorder the code a bit to be easier to parse.
* | | | | test-copy: test copy_bytes()Zbigniew Jędrzejewski-Szmek2015-06-101-0/+35
| |/ / / |/| | |
* | | | sd-bus: remove ucred parameter from bus_message_from_header() since we don't ↵Lennart Poettering2015-06-106-26/+3
| | | | | | | | | | | | | | | | use it anymore
* | | | sd-bus: fix early exit when we lack all data in bus_get_owner_creds_dbus1()Lennart Poettering2015-06-101-1/+1
| | | |
* | | | build-sys: upgrade shadow variable warnings to errorsLennart Poettering2015-06-101-0/+1
| | | |
* | | | bus-message: remove shadow warning with log_debug_bus_message()Lennart Poettering2015-06-101-13/+14
| | | |
* | | | Merge pull request #132 from ssahani/bondLennart Poettering2015-06-101-128/+45
|\ \ \ \ | | | | | | | | | | networkd: bond improve logging
| * | | | networkd: bond improve loggingSusant Sahani2015-06-101-128/+45
| | |_|/ | |/| | | | | | | | | | Replaces a lof of strerror() usage with log_netdev_error_errno()
* | | | Merge pull request #138 from ↵Lennart Poettering2015-06-101-12/+8
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | utezduyar/use-async-convenience-function-on-setting-hostname networkd: use async convenience call to set hostname
| * | | | networkd: use async convenience call to set hostnameUmut Tezduyar Lindskog2015-06-101-12/+8
| | | | |
* | | | | Merge pull request #133 from ssahani/netLennart Poettering2015-06-102-102/+34
|\ \ \ \ \ | | | | | | | | | | | | networkd: vxlan improve logging
| * | | | | network: veth imprve loggingSusant Sahani2015-06-101-18/+6
| | | | | | | | | | | | | | | | | | | | | | | | Replaces a lof of strerror() usage with log_netdev_error_errno()
| * | | | | networkd: vxlan improve loggingSusant Sahani2015-06-101-84/+28
| | |/ / / | |/| | | | | | | | | | | | | Replaces a lof of strerror() usage with log_netdev_error_errno()
* | | | | Merge pull request #137 from torstehu/correct-typoDaniel Mack2015-06-101-1/+1
|\ \ \ \ \ | | | | | | | | | | | | sd-bus: Correct typo
| * | | | | sd-bus: Correct typoTorstein Husebø2015-06-101-1/+1
|/ / / / /