test-strv.c: ported test_specifier_printf() to _cleanup_free_ + assert_se + cleanup

test-strv.c: added OOM check for current tests

New OOM check patch:

I do assert_se() before variable is used to do correct check.

test-strv.c: added strv_split_nulstr() test

tests: add test of memdup_multiply

man: Add reason for disk model info not working

In systemd-bootchart, the disk model information will not be found
unless the root device is specified using `root=/dev/sdxY` on the kernel
line. Just add a note as to why this doesn't happen.

bootctl: show "n/a" for empty boot option labels

udev: usb_id: parse only 'size' bytes of the 'descriptors' buffer

udev: usb_id - ignore non-ASCII serial numbers

On Thu, Feb 7, 2013 at 3:52 PM, Robert Milasan <rmilasan@suse.com> wrote:
> Hi, seems that using some strange usb devices with really bogus serial
> numbers usb_id creates links with junk strings in it:
>
> /dev/disk/by-id/usb-TSSTcorp_BDDVDW_SE-506AB_㡒䍌䜶䉗ぁㄴ㌴†ँ-0:0
>
> Initially was believed that usb_id is to blame, then the kernel, but it
> turns out that really the usb cd/dvd drive has this bogus serial number:
>
> output from dmesg:
> [  538.200160] usb 1-2: new high-speed USB device number 5 using
> ehci_hcd [  538.335067] usb 1-2: New USB device found, idVendor=0e8d,
> idProduct=1956 [  538.335080] usb 1-2: New USB device strings: Mfr=1,
> Product=2, SerialNumber=3 [  538.335089] usb 1-2: Product: MT1956
> [  538.335097] usb 1-2: Manufacturer: MediaTek Inc
> [  538.335105] usb 1-2: SerialNumber:
> \xffffffe3\xffffffa1\xffffff92\xffffffe4\xffffff8d\xffffff8c ...
> [  538.337540] scsi6 : usb-storage 1-2:1.0 [  539.341385] scsi 6:0:0:0:
> CD-ROM            TSSTcorp BDDVDW SE-506AB  TS00 PQ: 0 ANSI: 0
> [  539.354240] sr0: scsi3-mmc drive: 0x/24x writer dvd-ram cd/rw
> xa/form2 cdda tray [  539.354777] sr 6:0:0:0: Attached scsi CD-ROM sr0
> [  539.355122] sr 6:0:0:0: Attached scsi generic sg2 type 5

efi: compare by substraction

udev: expose new ISO9660 properties from libblkid

TODO

systemctl: allow comma sepearted property lists

man: prettify systemctl(1)

- reindent with two spaces, otherwise everything is squeezed near
  the right margin,
- mark only optional arguments as optional,
- some typos, etc.

bootchart: rename --filter to --no-filter

Turning off filtering with --filter is just too confusing.
Config option "Filter" doesn't have to be changed, here
"Filter=yes" already meant to filter.

bootchart: provide better error messages

man: move bootchart README to manpage, docbooksify

systemctl: Don't give re-activation warning if unit is masked

systemctl: allow status with no units

This makes 'status' behave like 'list-units':
systemctl status -> status of all units
systemctl -t error status -> status of error units
systemctl -t mount status -> etc.

unit: don't Require systemd-journald.socket from units

It is not really necessary to have a hard requirement dependency on
systemd-journald.socket in almost every unit. The socket gets pulled
into boot via at least two ways:
 sockets.target -> systemd-journald.socket
 sysinit.target -> systemd-journald.service -> systemd-journald.socket

So just assume something pulled the socket in and drop the automatic
requirement dependencies on it.

"systemctl stop systemd-journald.socket" will now not take the whole
system down with it.

execute: increase severity of journal connect failure message

journald is supposed to work. Failure to connect to its socket implies
losing messages. It should be a very unusual event. Log the failure with
LOG_CRIT.

Just because this unit's stdout/stderr failed to connect to the journal
does not necessarily mean that we shouldn't try to log the failure using
a structured entry, so let's use log_struct_unit.

execute: unify setup_{output,error}

The functions are quite similar. Unify them into one.

The source gets shorter, the binary gets slightly smaller.

execute: robustness against journald failures

Almost every unit logs to the journal. If journald gets a permanent
failure, units would not be able to start (exit code 209/STDOUT).

Add a fallback to /dev/null to avoid making the system entirely
unusable in such a case.

log: fix fallbacks to kmsg

write_to_journal() returns 0 if journal_fd is closed and nothing is
written. We need to make sure we'll try log_open_kmsg() then to make the
fallback work for "journal-or-kmsg".

man: Update bootchart.conf(5)

Recent changes in config options makes almost eveny variable in the man
page obsolete.

bootchart: if "root=/dev" is not matched, don't try to read /sys

update TODO

bootchart: stricter checks of cli input

bootchart: use conf-parser & CamelCase names in .conf

core: do not overwrite existing units source

Only set source for freshly created .mounts coming from
mountinfo file.

systemctl: print wall message only if successful

systemctl would write to the wall even if unsuccessful.

https://bugs.freedesktop.org/show_bug.cgi?id=60393

systemctl: use automatic cleanup

nspawn: fail if unable to close pipe

man: Add even more to the bootchart man page

Essentially transfer the 'README' file in src/bootchart to the man page
for easier reading by users.

Moved the 'History' section to the bottom, before the 'See Also' section.

 man/systemd-bootchart.xml | 100 ++++++++++++++++++++++++++++++++++++++--------
 1 file changed, 83 insertions(+), 17 deletions(-)

bootchart: parse /etc/os-release rather than system-release

Also parse it early, so that we can get it in the initramfs.

bootchart: more fixes for bootchart in the initramfs

nspawn: print PID and show how to enter the namespace

systemd-nspawn will now print the PID of the child.
An example showing how to enter the container is added
to the man page.

Support for nsenter without an explicit command was
added in https://github.com/karelzak/util-linux/commit/5758069
(post v2.22.2). So this example requires both a new kernel
and the latest util-linux.

added some missing include for a5c32cff1f56afe6f0c6c70d91a88a7a8238b2d7

honor SELinux labels, when creating and writing config files

Also split out some fileio functions to fileio.c and provide a SELinux
aware pendant in fileio-label.c

see https://bugzilla.redhat.com/show_bug.cgi?id=881577

bootchart items

man: use <replaceable> in various places

man: add filenames to the index

Limiting the addition to filenames from <refsynopsis>
seems to give a good enough S/N ratio.

man: bootchart: change /var/log to /run/log

As of [1], bootchart stores files in /run/log, not /var/log, by default.

[1] f2f85884caac671da84256acb44148df9a4dca70

Add systemd-analyze to .gitignore

Move systemctl dot to systemd-analyze dot

systemd-analyze: rewrite in C.

Written by Peeters Simon <peeters.simon@gmail.com>.
Makefile stuff and cleaned up a bit by Auke Kok <auke-jan.h.kok@intel.com>.
Some code inspired by Marc-Antoine Perennou <Marc-Antoine@Perennou.com>.

systemctl: use parse_unit_info

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

dbus: introduce parse_unit_info

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

dbus: import struct unit_info from systemctl

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

update TODO

build-sys: make EFI support build-time optional

build-sys: make PolicyKit support compile-time optional (was runtime-optional already)

build-sys: fix build without -Wl,--gc-sections

bootctl needs libsystemd-daemon.la too.

efi: efi_get_boot_options() should already sort the entries, the random order in the efivars fs is probably not useful

This also introduces a new FOREACH_DIRENT macro and makes use of it.

efi: use efi_get_variable_string() instead of efi_get_variable() where we can

update TODO

test-strv.c: Changed assert() to assert_se()

test-strv.c: Indent fix

test-strv.c: added strv_append test

test-strv.c: added strv_merge_concat test

test-strv.c: added strv_merge test

TODO

selinux: use a dynamically sized string when generating audit messages

https://bugzilla.redhat.com/show_bug.cgi?id=883043

use streq instead of strcmp

bootctl: always show slashes not backslashes

efi: unify BootXXXX reading

Revert "log_error() if inotify_add_watch() fails"

This reverts commit 2826d14091e43ed3397d862dee79d09d0115c84e.

We never should generate log messages from a library.

log_error() if inotify_add_watch() fails

[zj: Reworded message s/to watch/to add watch on/ to make it clear
     that it was the watch init action that failed, and not the
     "process of watching". I think this way it'll be clearer to
     people who don't know what inotify does.]

sysctl: fix uninitalized memory access in error path

src/sysctl/sysctl.c: In function ‘parse_file’:
src/sysctl/sysctl.c:172:41: warning: ‘property’ may be used uninitialized in this function [-Wmaybe-uninitialized]

man: rename systemd.conf to systemd-system.conf

Alias as systemd-user.conf is also provided. This should help
users running systemd in session mode.

https://bugzilla.redhat.com/show_bug.cgi?id=690868

man: Make options consistent

Option listings seemed to be pretty much random, some were short opt,
long opt, others were long opt, short opt. This just makes every option
with a short and long opt that I could find in the order short opt, long
opt, for formatting's sake.

man: Write man page for systemd-bootchart(1)

Mostly coppied from bootchart.conf(5)

test-udev: include missing.h

We need MS_REC and MS_SHARED defined for a proper compilation, so get them from
'missing.h' if not defined already.

ratelimit: fix off-by-one

The current code might make sense during startup for service restart
limiting:
With burst=1 after starting num is 1, so the next ('first') restart is
still accepted ( 1 <= 1). However, once interval has expired, num is 1
after the first restart, so the second restart is also accepted.

This change is also useful in combination with watchdogs:
With burst=1 and a very large interval the system can be restarted on
the first failure.

tests: add test for continuation lines with comments

build-sys: disable tests in po/, docs/ with separate build dir

Both gtk-doc and intltoolize have problems with VPATH builds.
"Creatively" disable tests when configuring from outside the
source directory.

This more-or-less reverts 9795da43c.

bootchart: Fix typos and some English

bootchart.conf: direct users to `bootchart.conf(5)`, not a nonexistent
man page.

svg.c: Fix some English in the svg comment header.
  * Chrome/Chromium => Chrome, Chromium
  * firefox => Firefox
  * much more slow => more slowly

man: typo fix

update TODO

util: *DO NOT* loop for EINTR handling with close_nointr()

See the linked references for why we should not do this.

boot: add stub Makefile symlink

TODO: bootchart items

bootchart: make bootchart work from within the initrd

With this patch, bootchart can be started from within the initramfs via
the kernel command line "rdinit=/usr/lib/systemd/systemd-bootchart"

see:
    http://harald.fedorapeople.org/downloads/bootchart-20130207-1652.svg

journal: Don't use loginuid if it's not valid

Code above this attempted to load loginuid, if this failed for
whatever reason, we'd still end up using that value (0) in place of
realuid.  Fix this by setting a bool when we know the loginuid is
valid.

This fixes journal messages showing up in per-user journals in
gnome-ostree (not configured with loginuid, but I'll shortly fix
that).

util: introduce strcaseeq/strncaseeq

use strneq instead of strncmp

Merge branch 'master' of ssh://git.freedesktop.org/git/systemd/systemd into work

efi: add missing #include

manager: pass environment over daemon-reexec

Fixes this bug:
alxchk > systemctl --user set-environment A=B
alxchk > systemctl --user show-environment | grep ^A=
A=B
alxchk > systemctl --user daemon-reexec
alxchk > systemctl --user show-environment | grep ^A=
alxchk >

systemd-bootchart: relicense to LGPLv2.1+

When we merged systemd-bootchart we omitted relicensing this
to LGPL. Now that code is being used for other purposes we
need to make sure the code is shareable within systemd
without further problems.

The original committers and contributors of bootchart before it
was merged all have agreed with the relicensing.

update TODO

efi: various cleanups

missing: define MS_REC and MS_SHARED if not defined already

udev: use unique names for temporary files created in /dev

On Tue, Feb 12, 2013 at 2:18 PM, Robert Milasan <rmilasan@suse.com> wrote:
> Under some circumstances udev mixed with multipath fails:
>
> udevd-work[1376]:
> symlink(../../sdk, /dev/disk/by-id/scsi-36005076305ffc0670000000000002842.udev-tmp)
> failed: File exists udevd-work[1432]:
> rename(/dev/disk/by-id/scsi-36005076305ffc0850000000000000a88.udev-tmp, /dev/disk/by-id/scsi-36005076305ffc0850000000000000a88)
> failed: No such file or directory
>
> This is non-fatal, but there is no point of created the symlink or
> renaming the symlink if it already exists.
>
> Reference: https://bugzilla.novell.com/show_bug.cgi?id=791503

It looke like this now:
  stat("/dev/disk/by-id", {st_mode=S_IFDIR|0755, st_size=80, ...}) = 0
  symlink("../../sda", "/dev/disk/by-id/ata-INTEL...N.tmp-b8:0") = 0
  rename("/dev/disk/by-id/ata-INTEL...N.tmp-b8:0", "/dev/disk/by-id/ata-INTEL...N") = 0

cdrom_id: add data track count for bad virtual drive implementations

/# /lib/udev/cdrom_id --debug /dev/sr0
probing: '/dev/sr0'
INQUIRY: [AMI     ][Virtual CDROM   ][1.00]
GET CONFIGURATION failed with SK=5h/ASC=20h/ACQ=00h
drive is pre-MMC2 and does not support 46h get configuration command
trying to work around the problem
READ DISC INFORMATION failed with SK=5h/ASC=20h/ACQ=00h
no current profile, but disc is present; assuming CD-ROM
READ TOC: len: 12, start track: 1, end track: 1
last track 1 starts at block 0
READ DISC INFORMATION failed with SK=5h/ASC=20h/ACQ=00h
ID_CDROM=1
ID_CDROM_MEDIA=1
ID_CDROM_MEDIA_CD=1

What is missing here is ID_CDROM_MEDIA_TRACK_COUNT_DATA to trigger
blkid in /lib/udev/rules.d/60-persistent-storage.rules

KERNEL=="sr*", ENV{DISK_EJECT_REQUEST}!="?*",
ENV{ID_CDROM_MEDIA_TRACK_COUNT_DATA}=="?*",
ENV{ID_CDROM_MEDIA_SESSION_LAST_OFFSET}=="", \
  IMPORT{builtin}="blkid --noraid"

localectl: fix --help text

https://bugs.freedesktop.org/show_bug.cgi?id=60595

journalctl: allow both "-n 55" and "-n55" on the command line, as equivalent syntaxes

https://bugs.freedesktop.org/show_bug.cgi?id=60596

manager: clean environment before passing it on to others

binfmt,tmpfiles,modules-load,sysctl: rework the various early-boot services that work on .d/ directories

This unifies much of the logic behind them:

- All four will now ofllow the rule that the earlier file and earlier
  assignment in the .d/ directories wins. Before, sysctl was the only
  outlier, where the later setting always won.

- All four now support getopt() and --help on the command line.

- All four can now handle specification of configuration file names on
  the command line to apply. The tools will automatically find them, and
  apply them. Previously only tmpfiles could do that. This is useful for
  %post scripts in RPMs and suchlike.

- This fixes various error path issues in conf_files_list()

core: properly validate environment data from Environment= lines in unit files

manager: validate environment parameters for SetEnvironment(), UnsetEnvironment() bus calls

bootctl: print partition uuids as /dev/disk/by-partuuid/...