libmount: more robust options string parsing

 # mount -o=rw /dev/sdb /mnt/test
 mount: libmount/src/optmap.c:212: mnt_optmap_get_entry: Assertion `namelen' failed.
 Aborted (core dumped)

Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=968786
Signed-off-by: Karel Zak <kzak@redhat.com>

agetty: don't remove ECHOCTL from c_lflag

It's really unexpected that arrows keys move cursor when agetty/login
asks for login name or password.

Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=969458
Signed-off-by: Karel Zak <kzak@redhat.com>

agetty: update man page and usage()

 - baud_rate is optional
 - agetty has been rewritten 2 years ago, so don't blame original
   authors in the man page

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

cal: fix -y output for UTF8

... in the yearly() function is a static buffer where is not space for
multibyte headers.

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

login: use TCSANOW to remove HUPCL

Fedora 19, kernel 3.9, systemd 204, telnetd with socket activation.

login(1) code:

        ttt.c_cflag &= ~HUPCL;
        tcsetattr(0, TCSAFLUSH, &ttt);

        close(STDIN_FILENO);
        close(STDOUT_FILENO);
        close(STDERR_FILENO);

        signal(SIGHUP, SIG_IGN);        /* so vhangup() wont kill us */
        vhangup();
        signal(SIGHUP, SIG_DFL);

strace :

ioctl(0, SNDCTL_TMR_CONTINUE or SNDRV_TIMER_IOCTL_GPARAMS or TCSETSF, {B9600 opost isig icanon echo ...}) = 0
close(0)                                = 0
close(1)                                = 0
close(2)                                = 0
--- SIGHUP {si_signo=SIGHUP, si_code=SI_KERNEL} ---
--- SIGTERM {si_signo=SIGTERM, si_code=SI_USER, si_pid=1, si_uid=0} ---
+++ killed by SIGTERM +++

It seems we get SIGHUP (and SIGTERM from systemd) before vhangup().
The problem occur sometimes.

I guess it's because TCSAFLUSH is not applied immediately, so it would
be probably better to use TCSANOW to make the code more robust.

References: https://bugzilla.redhat.com/show_bug.cgi?id=962145
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: release++ (v2.23.1)

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

docs: update v2.23.1-ReleaseNotes

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

po: merge changes

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

po: update zh_CN.po (from translationproject.org)

po: update vi.po (from translationproject.org)

po: update nl.po (from translationproject.org)

tests: new warning fix

2.23 added -Wstrict-prototypes check. Fix make check showing
CC tests/helpers/test_md5.o
tests/helpers/test_md5.c:8: warning: function declaration isn't a prototype

Signed-off-by: Gilles Espinasse <g.esp@free.fr>

lib/tty: fix get_terminal_name() usage, cleanup code

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

build-sys: always check for setns() and unshare() libs functions

 $ ./configure --disable-unshare
 $ make nsenter

  CC       sys-utils/nsenter.o
In file included from sys-utils/nsenter.c:36:0:
./include/namespace.h:31:19: error: static declaration of ‘unshare’ follows non-static declaration
In file included from /usr/include/sched.h:42:0,
                 from sys-utils/nsenter.c:23:
/usr/include/bits/sched.h:86:12: note: previous declaration of ‘unshare’ was here
make: *** [sys-utils/nsenter.o] Error 1

We have to always check for the libc function independently on
--disable-{unshare,nsenter} to avoid collision between local and libc
declarations.

Reported-by: "Gabor Z. Papp" <gzp@papp.hu>
Signed-off-by: Karel Zak <kzak@redhat.com>

sfdisk: don't call BLKRRPART for non-block devices

Reported-by: Bertrand Jacquin <beber@meleeweb.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

chcpu: clean up error messages

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

sulogin: clean up error messages

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

libblkid: add ReFS magic string

Reported-by: "Richard W.M. Jones" <rjones@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

blkid: fix typo in ID_FS_AMBIVALEN=

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

libmount: don't ignore root fs on umount -a

The release v2.22 (the new umount) introduces a regression when root
fs is excluded from umount --all. There is not reason for this
exception. The libmount should be smart enough to disable mtab update
after rootfs umount.

Reported-by: Bruce Dubbs <bruce.dubbs@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

lib/tty: don't hardcode terminal fd in get_terminal_name()

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

standard SEE ALSO section of man pages

The standard format is to seperate each entry with a comma, and
for each one to be on a line by itself.  Most util-linux pages
follow this, but a few do not.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

lsblk: add SERIAL column

 ... to return short serial number of the disk (from udev db)

Addresses: https://github.com/karelzak/util-linux/issues/33
Signed-off-by: Karel Zak <kzak@redhat.com>

losetup: use warn_size() for regular files only

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

lib/loopdev: fix loopcxt_check_size() to work with blkdevs

The loopcxt_check_size() is workaround for kernels < v3.9, kernel has
been fixed by commit 541c742a7559eb65f0e36d3e2338c2ca532a3e61.

The function sets loopdev size according to backing file size. The
problem is that the backing file could be a block device where
stat.st_size is zero, so we have to use blkdev_get_size() for block
devices.

Addresses: https://bugs.archlinux.org/task/35193
Reported-by: Dave Reisner <d@falconindy.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libfdisk: do not use va_list in the Ask API

  # fdisk /dev/sda
  Welcome to fdisk (util-linux 2.23).

  [...]

  Command (m for help): a
  Segmentation fault (core dumped)

It's too fragile, the va_list implementation is architecture specific
and it seems we need such thing in libfdisk at all.

Reported-by: "Gabor Z. Papp" <gzp@papp.hu>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: use MS_SILENT for /{proc,etc}/filesystems

mount(8) needs to be doing silent mounts when doing this brute
forcing or when the filesystem is not explicitly specified.

Reported-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: use -Wno-clobbered for some BSD crap

 * extend UL_WARN_ADD() for accept non-default CFLAGS variable
 * add BSD_WARN_CFLAGS with -Wno-clobbered
 * use all this for pg(1) and more(1)

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

libblkid: fix blkid_parse_tag_string() to ignore incomplete tags

fstab entry:

    FOO=     /       btrfs  defaults       1 1

Program received signal SIGABRT, Aborted.
0x00007f016ccd8819 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:56
56   return INLINE_SYSCALL (tgkill, 3, pid, selftid, sig);
(gdb) bt
    at ../sysdeps/unix/sysv/linux/libc_fatal.c:196

Reported-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: add --disable-setterm

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

build-sys: release++ (v2.23)

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

docs: update v2.23-ReleaseNotes

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

docs: update AUTHORS file

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

po: merge changes

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

po: update uk.po (from translationproject.org)

po: update pl.po (from translationproject.org)

po: update nl.po (from translationproject.org)

po: update hr.po (from translationproject.org)

po: update fr.po (from translationproject.org)

po: update da.po (from translationproject.org)

po: update cs.po (from translationproject.org)

tests: make mount tests backwardly compatible with deprecated-mount/

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

mount: (deprecated) check loopcxt_ return values

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

mount: (deprecated) remove duplicate xstrndup()

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

build-sys: use 'check' for all utils with dependencies

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

AUTHORS: fix prlimit typo

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

libfdisk: gpt: remove leftover comments

I must have left those in some commit.

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

rtcwake: pass -h to shutdown

The sysvinit shutdown requires you use -h when using -H or -P.
Other inits just don't seem to care.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

build-sys: fix non-linux build

Make enabling fdformat depend on being a linux build.

Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>

dmesg: minor cleanup of EXCL_xxx_ERR macros

Delete EXCL_ACT_ERR and EXCL_SYS_ERR defines:
usage is removed in 43d2eeef5dd3a64c372dbf60e4a32aa9e97b4645:
    "dmesg: use err_exclusive_options()"

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

rtcwake: add support for "freeze" mode

Supported by Linux since 3.9 (upstream commit
7e73c5ae6e7991a6c01f6d096ff8afaef4458c36).

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

umount: use devno to identify --all-target devices

 # mount -t tmpfs tmp /mnt/A
 # mount --bind /mnt/A /mnt/B
 # mount -t tmpfs tmp /mnt/C

the command:

  umount --all-targets /mnt/A

has to umount only A and B, but not C. The C is a another filesystem.

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

libfdisk: (gpt) fix EFI UUID usage

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

fdisk: fix list types

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

libfdisk: (gpt) fix valid_pmbr()

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

libfdisk: (gpt) use le64_to_cpu() for alternative LBA checks

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

libblkid: remove obsolete code

 ... forgot in the previous commit to the swap prober.

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

libblkid: ubifs is LE

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

libblkid: make swap detection arch independent

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

tests: fix compiler warning

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

build-sys: use pkg-config results for libuser

Our code depends on libuser >= 0.58, it's better to completely rely on
pkg-config results rather than also call UL_CHECK_LIB (that overwrites
previous pkg-config have_user= result independently on libuser
version).

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

build-sys: fix UL_REQUIRES_HAVE

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

fdisks: fix compiler warning

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

sulogin: fix compiler warnings

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

build-sys: fix nsenter requirements

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

include: fix SYS_{unshare,nsenter} usage

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

lscpu: set 32-bit / 64-bit for ppc

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

libblkid: use more robust code in silicon_raid checksum prober

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

libblkid: fix silicon raid checksum code on BE

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

tests: add BE minix test

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

libblkid: arch independent minix detection

It seems that on-disk MINIX FS superblock is native-endian.

Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=833841
Signed-off-by: Karel Zak <kzak@redhat.com>

tests: consolidate scsi_debug usage

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

libmount: add debug message to guess FS function

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

tests: use udevadm settle rather than sleep

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

libmount: fix mount.nfs segfault, rely on assert() rather than on nonnull

We use
   mnt_optstr_append_option(&o, mnt_fs_get_vfs_options(fs), NULL);

in mount.nfs, unfortunately mnt_optstr_append_option() has been marked
ass nonnull(1, 2). That's wrong because append and prepend should
robust enough to accept NULL as option name.

The patch also removes almost all __attribute__((nonnull). It seems
better to rely on assert() to have usable feedback. In many cases
(nonnull) is premature optimization for the library. This attribute
makes sense for things like strlen() or so...

Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=948274
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: release++ (v2.23-rc2)

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

docs: update v2.23-ReleaseNotes

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

build-sys: make wall optional in bash-completion

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

build-sys: add --disable-fdformat

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

build-sys: add --disable-hwclock

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

build-sys: add --disable-bfs

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

build-sys: add --disable-cytune

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

docs: update AUTHORS file

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

po: merge changes

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

blkid: use xalloc everywhere

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

tests: make line(1) optional

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

libblkid: remove duplicate includes

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

lib/xalloc: add xstrndup()

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

tests: update lscpu tests

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

libblkid: fix ZFS superblok offset for SBMAGIC

 # wipefs /dev/sdd1

 offset               type
 ----------------------------------------------------------------
 0x65000              zfs_member   [raid]
                      LABEL: zm1
                      UUID:  18067835215330778964

 and "wipefs -o 0x65000 /dev/sdd1" seems to run in an endless loop.

Reported-by: Helmut Hullen <Hullen@t-online.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

bash-completion: add swapoff

[kzak@redhat.com: remove short options]

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

lscpu: add Model name

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

su: remove '+' from getopt_long()

 # /bin/su - -c 'echo test'
 su: user -c does not exist

Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=950495
Signed-off-by: Karel Zak <kzak@redhat.com>

loopdev: sync capacity after setting it

I recently tried to mount an hfsplus file system from an image file with
a partition table by using the loop offset and sizelimit options to specify
the location of the file system.

hfsplus stores some metadata at a set offset from the end of the partition,
so it's sensitive to the device size reported by the kernel.

It worked with this:

But failed with this:

/dev/loop0: [0089]:2 (<imagefile>), offset 32768, sizelimit 102400000
/dev/loop1: [0089]:2 (<imagefile>), offset 32768, sizelimit 102400000

/proc/partitions shows the correct number of blocks to match the sizelimit.

But if I set a breakpoint in mount before the mount syscall, I could see:
102400000
102432768

The kernel loop driver will set the gendisk capacity of the device at
LOOP_SET_STATUS64 but won't sync it to the block device until one of two
conditions are met: All open file descriptors referring to the device are
closed (and it will sync when re-opened) or if the LOOP_SET_CAPACITY ioctl
is called to sync it. Since mount opens the device and passes it directly
to the mount syscall after LOOP_SET_STATUS64 without closing and reopening
it, the sizelimit argument is effectively ignroed. The capacity needs to
be synced immediately for it to work as expected.

This patch adds the LOOP_SET_CAPACITY call to loopctx_setup_device since
the device isn't yet released to the user, so it's safe to sync the capacity
immediately.

[kzak@redhat.com: - port to the current git HEAD,
                  - use uint64_t]

Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: export blkid_init_debug()

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

libblkid: support LIBBLKID_DEBUG= only

 ... the BLKID_DEBUG= is not more supported.

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

libblkid: fix DBG()

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

docs: add fsck.cramfs manual page

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

docs: add mkfs.cramfs manual page

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

tools: make checkmans.sh to find missing manuals

Finding missing manuals is loosely based on ideas in man-page-day-1.sh,
wrote by Peter Schiffer.

CC: Peter Schiffer <pschiffe@redhat.com>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>