include: [xalloc.h]: mention strdup in the file description

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

lscpu: use xstrdup from xalloc.h

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

losetup: spelling fixes

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

hexdump: new usage(), xalloc and err.h stuff

New usage help screen and print version switch. Also fixes to
exit codes, util linux xmalloc replaced emalloc and every error
print is using libc error function.

[kzak@redhat.com: - minor changes in formatting and coding style]

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

ipcs: really show all resources when -a and -i are combined

When you have more than one resource with the same id (but differing
types) combining -a and -i does not show all resources. This patch
corrects that.

Signed-off-by: Jens Kristian Søgaard <jens@mermaidconsulting.dk>

lsblk: fix readlink() usage

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

Revert "libmount: do not check for EAGAIN after flock()"

 Linux is not 100% POSIX compatible in this area...
 strace: "flock(3, LOCK_EX|LOCK_NB)   = -1 EAGAIN (Resource temporarily unavailable)"

This reverts commit 78d8c28f746bcc8d4e642c39669d8e58dea8d029.

Revert "libuuid: do not check for EAGAIN after flock()"

 Linux is not 100% POSIX compatible in this area...
 strace: "flock(3, LOCK_EX|LOCK_NB)   = -1 EAGAIN (Resource temporarily unavailable)"

This reverts commit d6c8d399b93f4646a90d0180fdcc1be35235a9c7.

build-sys: add uuid_generate_time_safe.3 .gitignore

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

docs: update TODO file

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>

tests: [cpuset] call free() for range

The free() before exit() is not so important, but let's keep the
example code nice and consistent.

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: move __uuid function to UUIDD_PRIVATE in uuid.sym

The private functions for uuidd should not be included in the normal
versioned part or the library.

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

libuuid: update manpage, mention uuid_generate_time_safe

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

libuuid: introduce uuid_generate_time_safe()

The problem with libuuid interface is that it is impossible to
inform the caller of uuid_generate_time() if the UUID was generated
in a safe manner (either via uuidd, or using the global clock state
counter).

This patch introduces new function,

int uuid_generate_time_safe(uuid_t out)

which can report whether the generated UUID is safe.

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

libuuid: __uuid_generate_time(): report if clock_seq is safe

Propagate return value from get_clock() through __uuid_generate_time().

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

libuuid: rename uuid__generate_* to __uuid_generate_*

Having uuid_generate_time and uuid__generate_time at
the same time is confusing. These functions are exported,
but not part of the official API (used by uuidd only).

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

libuuid: get_clock(): report if the clock_seq is safe

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

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>

partx: improve clarity of some messages, gettextize two missed ones

[kzak@redhat.com: - use <disk> rather than <device> in
                    error/verbose messages]

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

textual: improve the wording of some error and usage messages

[kzak@redhat.com: - cleanup lscpu(1) usage text
                  - use <disk> rather than <device> in partx(8)
                    usage text]

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>
Signed-off-by: Karel Zak <kzak@redhat.com>

textual: fix three typos in message strings and improve consistency

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

fsck: use xmalloc, warn & err and new usage

All compler errors reported by -Wall -Wextra -pedantic are fixed.

The patch also makes fsck to use xalloc instead of it's own malloc.
That will make fsck to exit, which could be good thing, at sections
where it previously did not.

Almost all print outs where touched, where various printfs where
converted to warn or err messages.

Finally theres a fix to counter wrap bug that could (very unlikely)
manifest itself if an user specifies same flag exactly UINT_MAX times.

[kzak@redhat.com: - coding style
                  - define XALLOC_EXIT_CODE
                  - use verbose++ in main()]

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

docs: favour small patches

http://www.spinics.net/lists/util-linux-ng/msg04084.html

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

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>

more: new usage output, and some trivial fixes

Usage output screen got to be completely rewrote. Aim is to
assist user to determe which switches to use without reading
manual page. The change also introduced new option to see command
version.

Determination if the executable name is `page' is now easy to see
in source, and effect of using the name is mentioned in manual
page.

Rest of the change is trivial; exit values to compiler warning
removals, magic constants to preprocessor definitions, symbolic
exit values, comment pretty printing and #endif markups.

[kzak@redhat.com: - coding style
                  - add __unused__ attribute to signal handlers]

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

logger: support for logging to UDP socket / remote syslog server

It adds the ability to logger to log a message to a udp socket.  The -n option
followed by the hostname of the remote host is mandatory to do this.  The
optional -P option can be used to change the UDP destination port (default
514).  The function udpopenlog is used to open the udp socket. After that
everything works in almost the same way like it does when logging to a UNIX
socket.

Signed-off-by: Josef Wuebbels <josef.wuebbels@mtu.de>

build-sys: provide alternatives for err, errx, warn and warnx

Solaris lacks err, errx, warn and warnx.  This also means the err.h header
doesn't exist.  Removed err.h include from all files, and included err.h from
c.h instead if it exists, otherwise alternatives are provided.

Signed-off-by: Fabian Groffen <grobian@gentoo.org>

sfdisk: rename warn to my_warn

To avoid a conflict with the warn function from err.h, renamed warn to
my_warn, as the functionality of both is not the same.

Signed-off-by: Fabian Groffen <grobian@gentoo.org>

mkfs.minix: general cleanups

This patch provides some minor code fixes: add/remove some macros and close the device after usage.
Also modifies usage() and changes main()'s indentation to 8-characters.

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

docs: update TODO

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

include: [c.h]: add fallback for old libs without O_CLOEXEC

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>

libmount: more robust flock() usage

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

libuuid: fix locking of clock state counter file

fcntl(2) does not work for synchronization of threads belonging
to the same process. Use flock(2) instead.

http://marc.info/?l=util-linux-ng&m=129708412829971&w=2

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

schedutils: remove unneeded header files

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

lscpu: add description of --sysroot option to man page, plus grammar touches

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

lsblk: small grammar touches to the man page

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

partx: normalize the punctuation of man page, and improve grammar here and there

Signed-off-by: Benno Schulenberg <bensberg@justemail.net>

blkid: dynamically allocate devicename array

If more than 128 devices are specified on the blkid cmdline,
the devices[] array will overflow.

We can dynamically allocate the devices[] array based on number
of arguments to avoid this problem.

[kzak@redhat.com: - add "if (optind < argc)" check]

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

wipefs: use write_all from include/writeall.h

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

swapon: use EXIT_SUCCESS constants

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

dmesg: use strtol_or_err instead of atoi

We shouldn't be accepting things like 'dmesg -n 2crapinput'
This patch also changes the exit's value to use EXIT_* constants.

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

lscpu: cleanup usage()

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

uuidd: correctly report number of generated subsequent UUIDs

Before this change, "uuidd -t -n 1" reported
"6fd0672e-2f9b-11e0-b9c8-000feae00cb0 and subsequent 1 UUIDs".
That is two subsequent UUIDs, which is not correct.

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

uuidd: fix -n usage

It does not make sense to request zero UUIDs - report error.

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

uuidd: increase connection backlog size to SOMAXCONN

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

uuidd: use write_all from include/writeall.h

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

uuidd: use die() where possible

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

uuidd: use daemon(3) library function

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

uuidd: use UUID_STR_LEN and UUID_LEN

Use named constants for lentgh of UUID's binary and textual
representation, instead of magic numbers.

[kzak@redhat.com: - use sizeof(uuid_t) rather than hardcoded size]

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

uuidd: use UUIDD_OP_BULK_*_UUID named constants

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

uuidd: fix uuidd -r -n 2

uuidd -r -n 2 failed with "Unexpected reply length from server 36".
Fixed by this patch.

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

include: [xalloc.h] include string.h

xalloc.h needs string.h for declaration of strdup().  Otherwise we
can get implicit declaration warning.  This patch prevents it.

Signed-off-by: Marek Polacek <mpolacek@redhat.com>

include: [c.h] add fallback for alloc_size attributes

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

fsck: include c.h, fallback for PATH_MAX

Reported-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

include: [c.h] move ignore_result() to gcc-only block

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

libmount: add and copy FS internal flags

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

libmount: don't prepare update if syscall failed

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

docs: update ReleaseNotes

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

blkid: add new return code 8 for ambivalent low-level probing result

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

blkid: add note about PART_ENTRY_ tags to blkid.8

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

blkid: don't return PART_ENTRY_* if partition is empty

The PART_ENTRY_* has nothing to do with the _contents_ of the block
device. So if the device is empty these values should not be printed
and blkid(8) has to return 2.

The PART_ENTRY_* for empty devices should be printed for "-o udev"
output only.

Reported-by: Thomas Bächler <thomas@archlinux.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

blkid: don't print devname if nothing detected

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

libmount: cleanup helper initialization API

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

libmount: add low-level API for umount

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

libmount: cleanup high-level mount API

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

libmount: copy mount attrs from utab to context

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

libblkid: check for swap or DM cow on small devices

The 1MiB limit for swap and COW is useless for Fedora LiveDVD where is
938KiB COW device.

Reported-by: Milan Broz <mbroz@redhat.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: conditionalise parts from blkid/topology for Linux

Signed-off-by: Fabian Groffen <grobian@gentoo.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

build-sys: shell syntax fix in autogen.sh

Use of == as synonym of single = sign backfires on systems where
/bin/sh is dash, like with Ubuntu.

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

mount: fix parsing offset= followed by more options

mount does not parse parse offset= option if it is followed by other
options.  In umount, the parsing is done with the get_value() function.
This patch moves get_value to fstab.c (with new name get_option_value())
and fixes mount to use the function.

[kzak@redhat.com: - rename to get_option_value()
  - use fstab.c rather than sundries.c]

Novell bugzilla: #666150

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

build-sys: provide missing strnlen from strutils

Solaris lacks strnlen, provide it from strutils.c.

Signed-off-by: Fabian Groffen <grobian@gentoo.org>

swapon: make needlessly global variables static

Also constify the option array.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>