libmount: fix mtab update for "none" source, fix leak

tab_parse.c:mnt_parse_mountinfo_line parses "none" in src as NULL,

tab_update.c:fprintf_mtab_fs sets m1 to NULL instead of "none" and
returns -ENOMEM
tab_update.c:update_table says "write entry failed: Success", as errno
hasn't been set, and gotos to leave, leaving mtab not updated.

Addresses-debian-bug: 634871
Reported-by: Tomas Janousek <tomi@nomi.cz>
Signed-off-by: Karel Zak <kzak@redhat.com>

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

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

docs: update 2.19.1 ReleaseNotes

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

build-sys: non-linux fixup

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

build-sys: release++ (v2.19.1-rc1)

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

docs: add v2.19.1 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 fr.po (from translationproject.org)

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

tests: don't run mount/regfile test on old kernel

The /sys/.../loop/backing_file is available since 2.6.37.

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

tests: add test for RLIMIT_FSIZE issue

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

libmount: remove unused variable from test

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

tests: use libmount tab update tests for UID=0 only

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

mount: remove unused variable

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

libmount: block signals when update utab

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

libmount: more robust mtab and utab update (CVE-2011-1676, CVE-2011-1677)

http://thread.gmane.org/gmane.comp.security.oss.general/4374

Changes:

 - always use temporary file

 - use fflush() for the temporary file

 - check fprintf() return value

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

mount: use lock from mnt_update_table()

The mnt_update_table() is able to create a mtab lock now.

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

umount: block signals when umounting and updating mtab (CVE-2011-1676, CVE-2011-1677)

http://thread.gmane.org/gmane.comp.security.oss.general/4374

Based on CVE-2011-1089.

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

mount: use fflush() and temporary file for mtab updates (CVE-2011-1089)

http://thread.gmane.org/gmane.comp.security.oss.general/4374

Changes:

  - force mount(8) to use /etc/mtab.tmp file every time. The original
    code used the tmp file for remount/move operations only.

  - call and check fflush() return code for the tmp file

Note mount(8) blocks all signals when writing to mtab, so it's not
affected by SIGXFSZ and the mtab lock file is always removed.

This patch does not fix the same issue in umount(8) and libmount.

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

libblkid: remove test for non-zero head count in FAT superblock probe.

This change enables the auto-detection of the internal storage on a VTech
Kidizoom camera, which has a zero head count value in the superblock.

Microsoft documentation, "FAT: General Overview of On-Disk Format", states
that the sectors and heads fields are only relevant for media that have
a geometry.

The Linux kernel also does not require a non-zero head count, and can
manually mount the filesystem.

Signed-off-by: Nick Holloway <Nick.Holloway@pyrites.org.uk>

libmount: avoid redundant declaration of mnt_context_do_umount

Signed-off-by: Jan Engelhardt <jengelh@medozas.de>

readprofile: don't stop parsing at __init_end

Some architectures like ARM place __init_end before .text section.
If any function in .init section is hit while profiling, readprofile
stops at __init_end.  That means if we enable profiling at boot time,
the profiler probably hits init functions and readprofile does not
work well unless we reset profiling buffer with -r option.

Signed-off-by: Po-Yu Chuang <ratbert@faraday-tech.com>

mount: add note about blank lines to fstab.5

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

umount: support non-canonical devnames in mtab

Addresses: https://bugs.launchpad.net/ubuntu/+source/util-linux/+bug/755193
Signed-off-by: Karel Zak <kzak@redhat.com>

setterm: segfault with long dump file name

The patch will make too long path to be truncated to MAX_PATH
lenght. If user supplies too long file name that is caught at
fopen. Some users might find this unintuitive, so the altered
behavior will be mantioned in manual page.

Bug in earlier version was visible with a command bellow.

setterm -dump -file $(for i in $(seq 254); do printf x ; done)

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

rev: memory leak fix

Moving the malloc out of loop will make leak to disappear, and
the command might run few jiffie quicker when there are 1+N
arguments.

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

libmount: fix parsing of mountinfo from 2.6.39

The /proc/self/mountinfo file uses " - " field as a separator between
optional fields and next fields in the file. The '-' char could be
used in the fields (for example in UUIDs), so it's necessary to check
for whole " - " string rather than for '-' char only.

Reported-by: "Aneesh Kumar K. V" <aneesh.kumar@linux.vnet.ibm.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: support /run/mount rather than /dev/.mount

The /run directory should be preferred on distributions where this
directory exists.

http://thread.gmane.org/gmane.linux.redhat.fedora.devel/146976

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

lscpu: detect sun4{u,v} in /proc/cpuinfo for sparc64

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

lscpu: fix op-mode for /{sys,proc} dumps

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

lscpu: fix threads-per-core calculation

On Thu, Mar 24, 2011 at 01:45:34PM +0100, Jan Engelhardt wrote:
> On a 24-thread/6-core SPARC T1, lscpu would wrongly output "5
> threads per core".
>
> It seems that the 6c T1 is simply an 8c T1 where 2c are disabled
> (offering a lesser model for a lower price, and all that marketing
> fluff). So the machine description header of the 6c T1 reports 32
> threads, but only goes on to provide 24 elements thereafter, which
> is why Linux will report threads 24-31 as "offline". So far so good.
>
> But lscpu would take the number of all (online and offline) threads
> (32) and divides it by the number of online cores (6), which yields
> an odd 5.33 threads/core.
>
> Simply pick the number of online threads.

Based on Jan's patch.

Reported-by: Jan Engelhardt <jengelh@medozas.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

login: fix checktty test to compile with old glibc

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

libmount: fix memory leak in cache

Reported-by: Petr Uzel <petr.uzel@suse.cz>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: small lock code cleanup

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

libmount: block signals when writing to mtab

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

blkid: small fix for safe_print()

This small fix makes it possible to print the first unprintable character
with code 128 from ASCII table.

Signed-off-by: Sergey Gusarov <laborer2008@gmail.com>

umount: use UMOUNT_NOFOLLOW for non-root users

The UMOUNT_NOFOLLOW umount2() flag is supported since kernel 2.6.34.

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

include: [c.h] add definition wrappers for old libc versions

Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: create a default lock for mtab update

Now it's necessary to explicitly create a lock for mnt_update_table().
It seems better to create the lock in mnt_update_table() by default to
skip this extra step.

Reported-by: Jan Engelhardt <jengelh@medozas.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

ddate: remove non-ascii chars from ddate

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

tailf: support --lines 0

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

fdisk: check index before access to array [coverity scan]

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

libblkid: use blkid_probe_get_buffer() more carefully

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

libmount: cleanup *_do_[u]mount() return codes and docs

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

tests: update GPT test

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

blkid: fix double free [coverity scan]

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

lib: [tt.c] dereferencing data before a null check [coverity scan]

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

tailf: fix open() return value check [coverity scan]

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

libmount: keep code more readable for analyzers [coverity scan]

This is not a bug ('ent' is always non-NULL is 'm' is non-NULL), but
let's keep static analyzes and humans who read the code happy.

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

libblkid: fix typo in *_to_cpu() usage in raid detection [coverity scan]

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

lsblk: fix strtoul() usage [coverity scan]

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

libblkid: add PART_ENTRY_* to docs

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

findmnt: check mnt_tab_next_fs() return code [coverity scan]

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

lsblk: use ssize_t for readlink() return code

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

blkid: don't ignore -i

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

include: [c.h]: typeof to __typeof__ fix

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

logger: fix variable type compiler warning

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

mount: update mtab on "-f" and compiled with libmount

Reported-by: silvan minghetti <bu1137@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

umount: segfaults with inconsistent entry in /etc/fstab

Addresses: https://bugs.launchpad.net/ubuntu/+source/util-linux/+bug/726283
Reported-by: Greg Brockman <gdb@gregbrockman.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: fix gcc warnings

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

libblkid: fix EFI GPT uuid byte order

Intel uses little-endians for UUID, the rest of the sane world uses 16
byte big-endian array...

Reported-by: Andrew Lutomirski <luto@mit.edu>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: try to detect if PT is newer than LVM

LVM (pvcreate) wipes the begin of the device. If there is a PT in the
wiped area then LVM signature is obsolete.

 # pvcreate /dev/sdb
 # fdisk /dev/sdb

old version:
 # blkid -p -o udev /dev/sdb
 ID_FS_TYPE=LVM2_member

new version:
 # blkid -p -o udev /dev/sdb
 ID_PART_TABLE_TYPE=dos

Reported-by: Matej Cepl <mcepl@redhat.com>
Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=679799
Signed-off-by: Karel Zak <kzak@redhat.com>

lsblk: bitwise or TT_FL_ASCII with tt_flags instead of setting

Signed-off-by: Dave Reisner <d@falconindy.com>

scriptreplay: bug in argument check

Number of arguments cannot be at the same time too many and few.

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

script: fix compiler warnings

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

lib: [strutils.c] more robust strtol checks

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

tests: add tests for [u]mount regular file

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

umount: allow unmounting loopdev specified by associated file

Make it possible to unmount a filesystem on a loop device if it is
specified by associated backing file. It does not attempt to unmount
anything if there are more than one loop device associated with the
given file.

Umount looks for associated loopdevice(s) only if umount is called
with the regular file as an argument.

Before:
mount -o loop -t ext2 img mnt
umount -v img
> Could not find /home/puzel/upstream/util-linux/img in mtab
> umount: img: not mounted

After:
mount -o loop -t ext2 img mnt
umount -v img
> img is associated with /dev/loop0, trying to unmount it
> /dev/loop0 has been unmounted

[kzak@redhat.com: - fix memory leak in lomount.c]

Addresses: https://bugzilla.novell.com/show_bug.cgi?id=666161
Signed-off-by: Petr Uzel <petr.uzel@suse.cz>
Signed-off-by: Karel Zak <kzak@redhat.com>

lsblk: fix readlink() usage

Reported-by: Steve Grubb <sgrubb@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

chsh: fix gcc link() warn_unused_result warning

setpwnam.c:176:9: warning: ignoring return value of ‘link’, declared with attribute warn_unused_result

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

fsck: fix -C parsing

Reported-by: Steve Grubb <sgrubb@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

chsh: fix small memory leak

Reported-by: Steve Grubb <sgrubb@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

login: does not ignore setgid() return code for non-roots

Reported-by: Steve Grubb <sgrubb@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: more robust whole-disk detection

In lib/wholedisk.c, i is set to 0. If fd was a -1, then at
line 18 geometry.start is used without it being initialized.

Reported-by: Steve Grubb <sgrubb@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libuuid: do not use invalid file descriptor

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

libuuid: manpage spelling fixes

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

libuuid: fix test in get_clock()

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

uuidd: fix manpage: libuuid(3) does not exist

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

lib: [strutils] avoid integer overflow on large values

This is visible on a 2TB disk via lsblk, where a large partition
incorrectly displays as 1.171798692T instead of 1.8T. This is
corrected by using a uint64_t type instead of a simple int --
consistant with the type used in lsblk.c to represent the raw size in
bytes.

[kzak@redhat.com: - add EiB support
                  - add size_to_human_string() to regression tests]

Signed-off-by: Dave Reisner <d@falconindy.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: add fallbacks for old systems without umount2() syscall

Reported-by: Peter Breitenlohner <peb@mppmu.mpg.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

include: [at.h]: include c.h for PATH_MAX

Reported-by: Peter Breitenlohner <peb@mppmu.mpg.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

libmount: replace mkostmps() with more portable mkstemp()

Reported-by: Peter Breitenlohner <peb@mppmu.mpg.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: don't build lsblk on Linux without openat()

Reported-by: Peter Breitenlohner <peb@mppmu.mpg.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

partx, lsblk: fix gettext calls

Reported-by: Peter Breitenlohner <peb@mppmu.mpg.de>
Signed-off-by: Karel Zak <kzak@redhat.com>

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

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

docs: update v2.19 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 pl.po (from translationproject.org)

po: update nl.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)

include: [c.h]: add unused attribute to ignore_result()

unfortunately, gcc 4.6

void fn (void) {
int foo;
foo = bar ();
}

evaluates that foo is never used. The __attribute__((unused)) is
necessary...

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

tests: fix libmount test

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

libuuid: do not check for EAGAIN after flock()

As per flock() manpage, flock() does not set errno=EAGAIN.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

libmount: do not check for EAGAIN after flock()

As per flock() manpage, flock() does not set errno=EAGAIN.

Signed-off-by: Petr Uzel <petr.uzel@suse.cz>

mount: add rootcontext= for tmpfs on SELinux boxes

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

include: [c.h]: fix __alloc_size__ test

Reported-by: Francesco Cosoleto <cosoleto@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>