sulogin: remove consoles.c from libcommon

- move struct chardata to include/ttyutils.h
- move console.{h,c} to login-utils/sulogin-* (it's sulogin specific)
- fix sulogin and agetty includes

Signed-off-by: Karel Zak <kzak@redhat.com>

mkfs.minix: fsck:minix: fix compiler warnings

Signed-off-by: Karel Zak <kzak@redhat.com>

mkswap: fix compiler warnings

Signed-off-by: Karel Zak <kzak@redhat.com>

lslocks: fix compiler warnings

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: (gpt) fix compiler warnings

Signed-off-by: Karel Zak <kzak@redhat.com>

setterm: fix compiler warning

Signed-off-by: Karel Zak <kzak@redhat.com>

ipcs: fix compiler warnings, use 64bit time

Signed-off-by: Karel Zak <kzak@redhat.com>

hwclock: fix compiler warning

Signed-off-by: Karel Zak <kzak@redhat.com>

ipcs: make --human really sexy

Align the "size" columns to the right.

Signed-off-by: Karel Zak <kzak@redhat.com>

lib/ismounted: improve loopdev support

... to make it usable for situations when the same backing
file is mapped to more loop devices.

Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: improve test_blkid_devno, fix compiler warning [-Wunused-result]

Signed-off-by: Karel Zak <kzak@redhat.com>

lib/ismounted: basic support for loop devices

 # losetup -a /dev/loop0
 /dev/loop0: [2053]:1048578 (/home/fs-images/filesystems/ext2.img)

 # findmnt /dev/loop0
 TARGET    SOURCE     FSTYPE OPTIONS
 /mnt/test /dev/loop0 ext3   rw,relatime,data=ordered

old version:

 ./test_ismounted /home/fs-images/filesystems/ext2.img
 not mounted

new version:

 ./test_ismounted /home/fs-images/filesystems/ext2.img
 mounted on /mnt/test

Reported-by: Sami Kerola <kerolasa@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/loopdev: fix loopdev_find_by_backing_file()

Signed-off-by: Karel Zak <kzak@redhat.com>

docs: add missing exit value to fsck.minix(8)

And remove references to manual pages which has not been part of commonly
used distributions for a while.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

minix: replace magic constants with macro names

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

minix: use off_t, size_t, etc appropriate types

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

minix: change types in header file

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

tunelp: deprecation: remove command from default build

It is difficult to think think tunelp command being in use a lot.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

more: remove unnecessary variable

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

isosize: inform if file does not look like iso file system

$ isosize /dev/urandom
isosize: /dev/urandom: might not be iso file system
-67690643227260

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

isosize: fix usage() -h option print out

Fixes usage() bug from commit 4eba43a7 which claimed sort option for
--help being -H.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcs: add --bytes size output option

This makes the command being a little closer standard compliant.  See
IEEE Std 1003.1 referral link below for more information.

References: http://pubs.opengroup.org/onlinepubs/009696799/utilities/ipcs.html
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcs: add --human readable size conversion option

Introduces new function ipc_print_size() which will call
size_to_human_string(), and handles the occasional '([k]bytes)' printing
if default size format is requested.

Reviewed-by: Karel Zak <kzak@redhat.com>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

ipcs: assist debugging

One will be able to see enum symbols when debugging, which is not the
case when values are specified as define list.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: swapon.8 option name fix

The commit 7ae8b469adda598fda28e4ea647d47905b43e172 added option --bytes,
not the initial proposal which was changed after maillist discussion.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

partx: add update command

Update the kernel partition table to match what is on disk.  Remove any
extra partitions and add any missing ones.

Signed-off-by: Phillip Susi <psusi@ubuntu.com>

sulogin: map NL to CR-NL on output in initial termios settings

(See also 9c62a2326e6e39a5e759830b0b9d91696166fb45 agetty patch).

Signed-off-by: Karel Zak <kzak@redhat.com>

sulogin: minor coding style changes

Signed-off-by: Karel Zak <kzak@redhat.com>

sulogin: add multi console feature from SysVinit sulogin

Now after adding Conflicts=rescue.service to getty@.service and
serial-getty@.service and Conflicts=getty.target to rescue.target
all works with `systemctl rescue'.  Even adding init=/sbin/sulogin
to the kernels command line by using the `e' key in grub2 boot
menu works flawless.

This version is without the emergency code as I've this moved to
lib/console.c where it belongs to.

Signed-off-by: Werner Fink <werner@suse.de>

sulogin: mount temporary /dev and /proc if not found

This is very usefull if initrd can not loaded that is no /dev and no
/proc is found.  Also if the /etc/shadow and /etc/passwd is copied into
the initrd the sulogin can be used in initrd even before /dev and/or /proc
are mounted.

Signed-off-by: Werner Fink <werner@suse.de>

sulogin: use alarm function to indicate if a timeout occurs

A small change to use the alarm handler to indicate if a timeout occurs.

Signed-off-by: Werner Fink <werner@suse.de>

sulogin: make usleep() workaround work

Simply fix a compile problem found during debugging console.c

Signed-off-by: Werner Fink <werner@suse.de>

sulogin: use the linked lists from list.h for consoles list

with this the double linked list feature from util-linux is used
instead of the single linked list.

[kzak@redhat.com: use list_last_entry() and list INIT macros]

Signed-off-by: Werner Fink <werner@suse.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

include/list: add list_last_entry()

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: remove crappy bugs section from man page

Signed-off-by: Karel Zak <kzak@redhat.com>

include/list: fix undefined behavior in list_entry() macro

Update list_entry() macro, which is basically the same as the
container_of() macro in the kernel, to use offsetof() to fix undefined
behavior.

Caught using clang -fsanitize=undefined.

[kzak@redhat.com: port from e2fsprogs]

Signed-off-by: Sami Liedes <sami.liedes@iki.fi>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: fix typo

Signed-off-by: Karel Zak <kzak@redhat.com>

docs: update AUTHORS file

Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: fix typos

Signed-off-by: Karel Zak <kzak@redhat.com>

blkid: fix double free [coverity scan]

Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: make mkstemp() more robust [coverity scan]

Signed-off-by: Karel Zak <kzak@redhat.com>

swapon: fix memmory leak [coverity scan]

Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: check mkdir(/run/blkid) return code [coverity scan]

Signed-off-by: Karel Zak <kzak@redhat.com>

swapon: don't care about xasprintf() return code

Signed-off-by: Karel Zak <kzak@redhat.com>

prlimit: don't care about xasprintf() return code

Signed-off-by: Karel Zak <kzak@redhat.com>

lscpu: don't care about xasprintf() return code

Signed-off-by: Karel Zak <kzak@redhat.com>

namei: don't care about xasprintf() return code

Signed-off-by: Karel Zak <kzak@redhat.com>

lslocks: don't care about xasprintf() return code

Signed-off-by: Karel Zak <kzak@redhat.com>

lsblk: don't care about xasprintf() return code

Signed-off-by: Karel Zak <kzak@redhat.com>

findmnt: don't care about xasprintf() return code

Signed-off-by: Karel Zak <kzak@redhat.com>

partx: don't care about xasprintf() return code

Signed-off-by: Karel Zak <kzak@redhat.com>

login: check lastlog lseek return code [coverity scan]

Signed-off-by: Karel Zak <kzak@redhat.com>

chsh: use return rather than exit in main, clean up man page

Signed-off-by: Karel Zak <kzak@redhat.com>

chfn: always use return rather than exit() in main

Signed-off-by: Karel Zak <kzak@redhat.com>

chfn: clean up exit status

The old version in some cases (but not always) returns -1 (255) on
error. It seems better to cleanup the code and don't return internal
return codes by exit().

Signed-off-by: Karel Zak <kzak@redhat.com>

tests: unset *_DEBUG variables

Reported-by: Bernhard Voelker <mail@bernhard-voelker.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: don't use safe_getenv() for test

Reported-by: Bernhard Voelker <mail@bernhard-voelker.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: avoid endless loop in mnt_get_kernel_cmdline_option

The above function infloops when the name to search for can only
be found at the beginning of /proc/cmdline but doesn't match,
e.g. when searching for "ro" in "root=/dev/sda1 quiet vga=0x31a".

* libmount/src/utils.c (mnt_get_kernel_cmdline_option): Replace
while by for loop to ensure the pointer p is incremented.

Signed-off-by: Bernhard Voelker <mail@bernhard-voelker.de>

rename: use macro to print version

Signed-off-by: Karel Zak <kzak@redhat.com>

rename: add --symlink option for renaming symlink target

[kzak@redhat.com: - coding style clean up]

Signed-off-by: Karel Zak <kzak@redhat.com>

blkid: fix man page mess

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: (gpt) check if the "First sector" is unused

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: (gpt) align newly created partitions

 - default "First sector" has to be aligned
 - "Last sector" has to be aligned if specified by +<size><suffix> convention

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: (gpt) fix {last,first}_usable_lba usage

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: cleanup partition start/end usage

 - always use pointers to make the code more generic
 - always use "const" is possible
 - always use le64_to_cpu() to access numbers

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: (gpt) check partitions alignment

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: report PT type, inform about new GPT

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: (gpt) use topology to define FirstUsableLBA

Signed-off-by: Karel Zak <kzak@redhat.com>

vipw: remove unused set{uid,gid} calls

Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: clean nonnull attribute usage

 - use __attribute__((nonnull) for functions where we not able to
   return an return code ("is", "has" and some "get" functions).

 - use __attribute__((nonnull) for small functions where we
   always modify any of the function argument (some mnt_optstr_* functions)

Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: re-read PART{UUID,LABEL} for successfully verified cached devices

Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: cleanup arguments checks

Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: cleanup nonnull attribute usage

 - don't mix __attribute__((nonnull)) and "if (!argument) return" for
   the same function. The "if ()" check will be *silently* optimized
   out by gcc.

 - use __attribute__((nonnull)) for functions where is impossible to
   return error code. For example "is" and "has" functions or functions
   where we return unsigned types and zero is a valid value.

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: move DOS units stuff to fdiskdoslabel.c

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: use BLKSSZGET as default for io-limits rather than hardcoded limit

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: remove global variable sector_offset

 - add cxt->first_lba as replacement to sector_offset
 - add label->reset_alignment() to overwrite the default alignment setting
 - add fdisk_reset_alignment() to setup grain, first LBA offset, etc.

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: zerosize EBR buffer

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: api: propagate add partitions to users

Similarly to commit 1f5eb51b79275e32d045fd6718753bf04cde8374 we
do not propagate problems when adding partitions to user visible
api.

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

fdisk: remove dead code

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

fdisk: api: move disklabel type to cxt

Get rid of it as a global variable as it belongs in the context.
To compare a disklabel on the device, the fdisk_dev_is_disklabel()
function is introduced. Also, to avoid naming issues, the fdisk_labeltype
members where renamed to FDISK_DISKLABEL_<type>.

Signed-off-by: Davidlohr Bueso <dave@gnu.org>

Merge branch '2012wk47' of git://github.com/kerolasa/lelux-utiliteetit

build-sys: fix typo

Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: use pkg-config for selinux

- we need libselinux 2.x (where is security_get_initial_context())
- the latest selinux versions are linked with -lpcre

Reported-by: Gregory Nietsky <gregory@distrotech.co.za>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: add devel-non-docs compile profile

Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: don't fails silently if EBR is outside of disk

Reported-by: Phillip Susi <psusi@ubuntu.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: cleanup blkid_strconcat

Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: remove blkid_{strndup,strdup}

Don't try to be smart. Let's use standard libc functions. (Note that
we have fallback for strndup() in include/strutils.h)

Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: add function attributes to private API

Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: add function attributes to public API

[kzak@redhat.com: - use __ul prefix in public API,
                  - define minimal requirement gcc 3.4]

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: add function attributes to private API

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: add function attributes to public API

[kzak@redhat.com: - use __ul prefix in public API,
                  - define minimal requirement gcc 3.4]

Signed-off-by: Ondrej Oprala <ooprala@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: cleanup TODO

Signed-off-by: Karel Zak <kzak@redhat.com>

lsblk: fix copy & past bug

Signed-off-by: Karel Zak <kzak@redhat.com>

umount: sanitize paths from non-root users

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: sanitize paths from non-root users

 $ mount /root/.ssh/../../dev/sda2
 mount: only root can mount UUID=17bc65ec-4125-4e7c-8a7d-e2795064c736 on /boot

this is too promiscuous. It seems better to ignore on command line
specified paths which are not resolve-able for non-root users.

Fixed version:

  $ mount /root/.ssh/../../dev/sda2
  mount: /root/.ssh/../../dev/sda2: Permission denied

  $ mount /dev/sda2
  mount: only root can mount UUID=17bc65ec-4125-4e7c-8a7d-e2795064c736 on /boot

Note that this bug has no relation to mount(2) permissions evaluation
in suid mode. The way how non-root user specifies paths on command
line is completely irrelevant for comparison with fstab entries.

Signed-off-by: Karel Zak <kzak@redhat.com>

lib/canonicalize: add canonicalize_path_restricted() to canonicalize without suid permisssions

Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: make debug stuff more robust

Signed-off-by: Karel Zak <kzak@redhat.com>

mount: (deprecated) drop --guess-fstype

The option is undocumented and unnecessary.

Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: correctly interpret '*' from /etc/filesystems

 - single line with '*' in /etc/filesystems means that libmount has to
   read /proc/filesystems, otherwise /proc/filesystems has to be ignored

 - mount(2) ENODEV is no reason to break the do_mount_by_pattern()
   loop when trying to mount by /{etc,proc}/filesystems

Reported-by: NeilBrown <neilb@suse.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: ddate removal

Missed in commit 4cf02b65109029a38f708148dea8d2bdf2fa19de

Signed-off-by: Sami Kerola <kerolasa@iki.fi>