summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* resolvectl: fix interface output when resolving hostnames→addressesLennart Poettering2018-12-051-9/+21
| | | | | | | | We already used in_addr_ifindex_to_string() which internally appends the ifindex to the address with % if necessary. It's simply wrong to attach the intreface a second time with % then. Also, it breaks stuff that cannot deal with that. Hence, let's reformat this, and add the ifindex as a comment to the output, and drop the second % suffix.
* resolvectl: remove allow_loopback parameter from ifname_mangle()Lennart Poettering2018-12-053-21/+13
| | | | | | | All our bus calls validate whether the specified device is a loopback device anyway on the server side. Let's hence simplify the client, there's no value in optimizing error paths after all. But there is value in simpler code.
* resolvectl: for our per-link verbs alternatively accept interface name in -iLennart Poettering2018-12-051-46/+65
| | | | | | | | | | | it's a bit confusing that we take two interfaces for verbs such as "dns" or "domain": once after the verb, and once as --interface=. While there's logic behind it, let's make this least surprising: if either is specified be happy. This means "resolvectl -i foo dns" is now equivalent to "resolvectl dns foo …". Note that this is a tweak only, to minimize surprises. We don't document this alternative syntax, and shouldn't to keep things simple.
* resolvectl: rework ifname_mangle()Lennart Poettering2018-12-051-26/+16
| | | | | | | | | | Let's compare the ifname passed in with what is set already if there is something set already. Complain in that case. This makes commands such as "resolvectl -i foo dns bar" less weird, as we'll refuse the duplicate ifname specifications. Also, free the old arg_ifname right before assigning the new, instead of doing so in advance.
* resolved: request incoming ifindex for DNS UDP packets, tooLennart Poettering2018-12-051-3/+19
| | | | | | | | | This information is useful to pass back to the caller, as it tells us where we found the answer. (While we are at it, fix the socket level for the RECVERR logic) Fixes: #9778
* Merge pull request #11053 from poettering/rc-local-tweakZbigniew Jędrzejewski-Szmek2018-12-051-11/+18
|\ | | | | rc-local-generator message tweak
| * rc-local-generator: tweak log message generationLennart Poettering2018-12-051-8/+17
| | | | | | | | | | | | | | | | | | Let's be more explicit, and not use LOG_WARNING log level when there's nothing really wrong. Fixes: #11044 Follow-up for: #10955
| * rc-local-generator: small shortificationLennart Poettering2018-12-051-3/+1
|/
* README: embed a CentOS CI badge into READMEEvgeny Vereshchagin2018-12-051-3/+4
| | | | Plus be more specific about where the badges point to.
* systemctl: drop my copyrightMarc-Antoine Perennou2018-12-051-3/+0
| | | | Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>
* Merge pull request #11025 from evverx/clang-asanEvgeny Vereshchagin2018-12-052-4/+25
|\ | | | | travis: also run the tests built with clang under ASan+UBsan
| * travis: install llvm to get llvm-symbolizerEvgeny Vereshchagin2018-12-041-1/+1
| | | | | | | | See https://clang.llvm.org/docs/AddressSanitizer.html#symbolizing-the-reports
| * travis: also run the tests built with clang under ASan+UBsanEvgeny Vereshchagin2018-12-042-3/+24
| |
* | Merge pull request #11051 from poettering/sock-addr-un-lenEvgeny Vereshchagin2018-12-051-13/+20
|\ \ | | | | | | trick out ubsan
| * | socket-util: Let's trick out ubsan when it validate struct ↵Lennart Poettering2018-12-041-4/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | sockaddr_un.sun_path[] Linux is stupid and sometimes returns a "struct sockaddr_un" that is longer than its fields, as it NUL terminates .sun_path[] even if it has full length. ubsan detects this, rightfully. Since this is a Linux misdesign let's trick out ubsan a bit. Fixes: #11024
| * | socket-util: break lines in sockaddr_pretty() signatureLennart Poettering2018-12-041-1/+7
| | | | | | | | | | | | It's soo long, let's make this more readable.
| * | socket-util: shorten code a bitLennart Poettering2018-12-041-8/+4
|/ /
* | man/daemon.xml: use AM_DISTCHECK_CONFIGURE_FLAGSOliver Smith2018-12-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | Change the man page to recommend setting AM_DISTCHECK_CONFIGURE_FLAGS instead of DISTCHECK_CONFIGURE_FLAGS. This is the recommended way from the automake manual, as otherwise the flag can't be overridden by the user with something like: DISTCHECK_CONFIGURE_FLAGS=--some-additional-flags make distcheck References: https://www.gnu.org/software/automake/manual/html_node/Checking-the-Distribution.html https://osmocom.org/issues/3718
* | rules: watch metadata changes on DASD devicesVojtech Trefny2018-12-041-1/+1
| | | | | | | | | | To make sure the change event is emitted and udev db is updated after metadata changes.
* | Merge pull request #11041 from yuwata/update-missing-v2Lennart Poettering2018-12-0450-745/+452
|\ \ | | | | | | missing: separate missing.h more
| * | util: drop missing.h from util.hYu Watanabe2018-12-0427-2/+27
| | |
| * | missing: move char{16,32}_t definitions to missing_type.hYu Watanabe2018-12-045-11/+16
| | |
| * | missing: move capabilities to missing_capability.hYu Watanabe2018-12-043-25/+14
| | |
| * | missing: move prctl related entries to missing_prctl.hYu Watanabe2018-12-043-40/+16
| | |
| * | missing: move statx related definitions to missing_stat.hYu Watanabe2018-12-044-44/+54
| | |
| * | missing: move entries defined in magic.h to missing_magic.hYu Watanabe2018-12-043-48/+36
| | |
| * | missing: move input related entries to missing_input.hYu Watanabe2018-12-043-33/+46
| | |
| * | missing: rename securebits.h to missing_securebits.hYu Watanabe2018-12-047-47/+19
| | |
| * | missing: move btrfs related entries to missing_btrfs.h and missing_btrfs_tree.hYu Watanabe2018-12-048-427/+135
| | |
| * | missing: move missing entries in keyctl.h to missing_keyctl.hYu Watanabe2018-12-044-68/+81
| | |
| * | meson: add missing missing_*.hYu Watanabe2018-12-041-0/+8
| | |
* | | Merge pull request #11042 from yuwata/tiny-coding-style-fixesLennart Poettering2018-12-0425-28/+28
|\ \ \ | | | | | | | | Tiny coding style fixes
| * | | tree-wide: add whitespace between type and variable nameYu Watanabe2018-12-0424-27/+27
| | | |
| * | | ip-protocol-list: sort headersYu Watanabe2018-12-041-1/+1
| |/ /
* | | cgroup: Use varname for cgroup_disable documentationChris Down2018-12-041-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | The current use of literal + replaceable is pretty ugly as it usually ends up with cgroup_disable= rendered in quotes, which looks really weird, and this doesn't conform with others of a similar type (for example, the earlier `DefaultDependencies=no` discussion in the same file.
* | | Merge pull request #11037 from poettering/json-tableZbigniew Jędrzejewski-Szmek2018-12-0414-46/+299
|\ \ \ | | | | | | | | optionally output tables as JSON
| * | | format-table: never try to reuse cells that have color/url/uppercase setLennart Poettering2018-12-031-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | The table cell reusing code is supposed to be an internal memory optimization, and not more. This means behaviour should be the same as if we wouldn't reuse cells.
| * | | tree-wide: specify all table headers in lower-caseLennart Poettering2018-12-036-11/+11
| | | |
| * | | format-table: add option to uppercase cells on displayLennart Poettering2018-12-032-12/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a per-cell option for uppercasing displayed strings. Implicitly turn this on for the header row. The fact that we format the table header in uppercase is a formatting thing after all, hence should be applied by the formatter, i.e. the table display code. Moreover, this provides us with the benefit that we can more nicely reuse the specified table headers as JSON field names, like we already do: json field names are usually not uppercase.
| * | | machinectl: optionally output table in JSONLennart Poettering2018-12-031-1/+7
| | | |
| * | | loginctl: optionally output table in jsonLennart Poettering2018-12-031-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | This works now: $ loginctl list-session --output=json-pretty
| * | | format-table: optionally covert/output table to JSONLennart Poettering2018-12-032-0/+167
| | | |
| * | | format-table: use right field for displayLennart Poettering2018-12-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Since .timespan and .timestamp are unionized on top of each other this doesn't actually matter, but it is still more correct to address it under it's correct name.
| * | | output-mode: add new helper OUTPUT_MODE_IS_JSON()Lennart Poettering2018-12-031-0/+4
| | | | | | | | | | | | | | | | | | | | Just a pretty way to determine whether the selected output mode is a JSON output mode.
| * | | output-mode: add generic helper to pick right JsonFormatFlags for given ↵Lennart Poettering2018-12-033-4/+22
| | | | | | | | | | | | | | | | OutputMode
| * | | json: add new display flag JSON_FORMAT_COLOR_AUTOLennart Poettering2018-12-033-7/+11
| | | | | | | | | | | | | | | | | | | | Typically we want to enable color when colors_enabled() says so, hence let's automatize this.
| * | | json: introduce explicit type for json formatting flagsLennart Poettering2018-12-032-8/+8
| | |/ | |/|
* | | Merge pull request #11009 from poettering/root-cgroup-againLennart Poettering2018-12-041-95/+74
|\ \ \ | | | | | | | | tweak root cgroup attribute fiddling for cgroupsv1 again
| * | | cgroup: suffix unit file settings with "=" in log outputLennart Poettering2018-12-011-6/+6
| | | | | | | | | | | | | | | | | | | | Let's follow our recommendations from CODING_STYLE and suffix unit file settings with "=" everywhere.
| * | | core: don't try to write CPU quota and memory limit cgroup attrs on root cgroupLennart Poettering2018-12-011-89/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the kernel sources attempts to write to either are refused with EINVAL. Not sure why these attributes are exported anyway on cgroupsv1, but this means we really should ignore them altogether. This simplifies our code as this means cgroupsv1 is more alike cgroupsv2 in this regard. Fixes: #10969