sfdisk: use fdisk_set_partition_type()

This is more simple than the generic fdisk_set_partition() API.

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

sfdisk: cleanup usage() and long options

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

sfdisk: add --parttype

The patch also makes --{id,change-id,print-id} deprecated in favour
of --parttype. The original --id is too generic option name and the
--print-id and --change-id are unnecessary and inconsistent with
another sfdisk options (e.g. we don't have --change-bootable)

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

sfdisk: add --verify

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

sfdisk: add --list-types

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

sfdisk: add missing stuff to usage()

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

sfdisk: make dump format usable on stdin

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

libfdisk: (gpt) use generic API to change UUID and name

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

libfdisk: (gpt) implement fdisk_set_partition() backend

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

libfdisk: (dos) implement fdisk_set_partition() backend

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

libfdisk: add fdisk_set_partition()

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

libfdisk: cleanup internal drivers' API

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

sfdisk: support -N for primary partitions

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

libfdisk: add fdisk_copy_parttype()

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

sfdisk: require confirmation before write to the device

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

libfdisk: make yes/no dialogs more robust

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

sfdisk: improve UI

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

libfdisk: cleanup script size= code

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

libfdisk: return partno when add new partition

 * improve the way how sfdisk report results
 * the API change simplify applications

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

sfdisk: implement command_fdisk()

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

libfdisk: fix script partitions size parsing

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

libfdisk: (dos) more carefully check for primary partitions

  * check if there is space for primary partitions, if no and extended
    exists then create logical partitions

  * create extended if @pa->type is extended partition

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

sfdisk: cleanup comments and command_activate()

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

sfdisk: add --activate

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

libfdisk: add fdisk_partition_is_bootable()

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

sfdisk: add comments, remove unnecessary function

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

sfdisk: add --show-size

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

fdisk: refactor /proc/partitions usage

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

sfdisk: update usage()

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

sfdisk: add --dump functionality

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

sfdisk: --list just one device

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

sfdisk: implement --list

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

fdisk: improve --list output

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

fdisk: move --list functionality to separated file

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

sfdisk: basic main() and friends

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

tests: add sfdisk test

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

libmount: fix mnt_is_readonly() #ifdef

This issue affects util-linux portability to GNU/HURD

Reported-by: Pino Toscano (from Red Hat)
Signed-off-by: Karel Zak <kzak@redhat.com>

libblkid: zeroize errno on blkid_probe_get_buffer() success

Since 37f4060225df0591ab8e1dd676dbc8115d900d4f prober functions are
sensitive to errno, it seems more robust to set errno=0 with in
blkid_probe_get_buffer() on success than set the zero on all places
where we call blkid_probe_get_buffer().

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

flock: zero timeout is valid

This patch reverts Sami's "timeout cannot be zero", introduced
in commit 605325b23b36238c8f3ae165e37cab9064553cf7.

The --timeout 0 has been originally interpreted as --nonblock. The
patch also add hint about this behavior to the man page.

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

losetup: improve man page -D docs

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

libfdisk: fix add_logical() assert

The zero index is absolutely valid for extended partition (it means
extended partition could be the first partition on the device).

Reported-by: Christoph Hoopmann <christophhoopmann@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

docs: add swapon -o to TODO file

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

hwclock: use fabs() instead abs() when argument is double

sys-utils/hwclock.c:1219:13: warning: using integer absolute value function
      'abs' when argument is of floating point type [-Wabsolute-value]

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

include/debug: remove duplicate 'const' declaration warning

./include/debug.h:89:55: warning: duplicate 'const' declaration specifier [-Wduplicate-decl-specifier]

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

lsblk: fix compiler warning

warning: ISO C forbids empty initializer braces [-Wpedantic]

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

include: fix compiler warnings

This change has motivation to make -Wall -pedantic compiler options to
spit a little bit less noise, which with luck will increase change to
notice real issues.

Multiple of these:

warning: ISO C forbids braced-groups within expressions [-Wpedantic]

And nearly 300 times reported:

include/c.h:75:41: warning: struct has no named members [-Wpedantic]
 #define UL_BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:-!!(e); }))

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

libuuid: remove _SVID_SOURCE pre-processor definition

The _SVID_SOURCE is deprecated.

In file included from /usr/include/stdio.h:27:0,
                 from libuuid/src/gen_uuid.c:46: /usr/include/features.h:148:3:
      warning: #warning "_BSD_SOURCE and _SVID_SOURCE are
      deprecated, use _DEFAULT_SOURCE" [-Wcpp]

Notice that the Makefile.am has

AM_CPPFLAGS = \
        -include config.h \

that enforces the config.h is included before any other hearer.  In the
configure.ac there is AC_USE_SYSTEM_EXTENSIONS that makes all possible
features that can be supported in a system are turned on.  With these two
there is absolutely no need for any legacy _SOURCE definitions.

Reference: https://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commit;h=c941736c92fa3a319221f65f6755659b2a5e0a20
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

textual: use usage() text element macros

Translating these text elements should happen only once, which is
more likely when the text macros are used properly.

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

textual: use manual tail usage() macro

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

textual: use version printing macro everywhere

Only mount, umount, and blkid remains not using the macro because they
are print also library references.

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

flock: add error message to translations

And remove type casting.

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

textual: fix couple typos in comments

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

hexdump: remove deprecation message

Message 'calling hexdump as od has been deprecated in favor of GNU
coreutils od' has informed the hexdump not to be used like that for three
and half years, and five releases.  It is time to get rid of notice.

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

docs: update deprecation banners

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

swapon: align available columns section in usage()

Commit 5db57cfc469b45ada0162a64d627ce25a3d2431f missed doing this change.

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

cfdisk: don't modify ncurses LINES and COLS

On Sun, Sep 28, 2014 at 01:48:38AM -0700, pv4 wrote:
 If ncurses is build with --enable-reentrant, building
 util-linux-2.25.1 fails with the folllowing error:

     disk-utils/cfdisk.c: In function 'resize':
     disk-utils/cfdisk.c:202:9: error: lvalue required as left operand of assignment
        LINES = ws.ws_row;
              ^
     disk-utils/cfdisk.c:203:8: error: lvalue required as left operand of assignment
        COLS = ws.ws_col;
             ^

 man COLS gives the following:

     Depending on the configuration, these may be actual variables,
     or macros (see curs_threads(3X)) which provide read-only
     access to curses's state.  In either case, applications should
     treat them as read-only to avoid confusing the library.

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

su: print proper core dump message when killed

An example is, in one terminal "sudo su -; echo $$", and in
another terminal, "kill -9  $PID" (the pid of the su -). It
should not print "(core dumped)", unless the kill signal
specified so, e.g. kill -7 or kill -11.

Signed-off-by: pcpa <paulo.cesar.pereira.de.andrade@gmail.com>

tests: fix the ipcs test for shmall being too big to show

Based on hints from Adam Sampson, Ruediger Meier and Sami Kerola.

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

libmount: don't return root if path contains // or ending /

If the path fx. is /foo/bar/ the initial stripoff will replace the last slash
with \0 and return a pointer to that exact \0 character. The same thing will
happen if the path contains // somewhere.

Signed-off-by: Søren Holm <sgh@sgh.dk>

textual: there may be no space between option and optional =arg

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

textual: angular brackets around a non-literal argument, singular

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

textual: grammarize another error message

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

textual: using angular brackets around individual arguments

And separate short and long options with the standard comma.

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

textual: grammarize an error message

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

textual: not using angular brackets around literal arguments

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

textual: add a comma for clarity, plus some other tweaks

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

textual: use a somewhat clearer and more standard failure message

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

textual: "ambivalent" applies to intentions, "ambiguous" to results

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

textual: use angular brackets around each individual argument

In usage texts each word that is an argument should be marked
separately with angular brackets.  Also add a translator comment.

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

lslogins: free after error [coverity scan]

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

lslogins: cleanup after error [coverity scan]

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

mount: improve --move docs

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

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

* 'misc' of git://github.com/kerolasa/lelux-utiliteetit:
  textual: share crypt() error message in sulogin and newgrp
  newgrp: avoid use of obsolete getpass() function
  newgrp: use libc function to read gshadow if it is available
  setarch: use personality() system call when it is available
  setarch: reindent code
  hwclock: remove referal to deprecated keyboard interface
  eject: make open_device() and select_speed() to use struct eject_control
  eject: add struct eject_control to remove global variables
  mountpoint: simplify if statement
  mkfs.minix: fix couple compiler warnings
  mountpoint: add struct mountpoint_control
  last: improve code readability by renaming variable names
  last: make is_phantom() when kernel config does not include audit support
  lib: remove xgetpass()
  include: simplify fputc_careful() in carefulputc.h
  libuuid: add extern qualifiers to uuid/uuid.h system header

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

* 'renice' of git://github.com/kerolasa/lelux-utiliteetit:
  renice: reorder usage() option descriptions
  rename: add getpriority() message lookup table
  renice: fix numeric uid argument parsing
  renice: avoid having same lines of code twice
  renice: disallow --priority <arg> without pid argument
  rename: use usage and version print out macros
  renice: reorder functions to avoid need of function prototype

lib/swapprober: add missing header file

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

swapon: split swapon-common.c

 swapon    - requires libmount and libblkid
 swapoff   - requires libmount
 swaplabel - requires libblkid

This patch add lib/swapprober.c with blkid stuff for swap. It allows
to use and link libblkid only when necessary.

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

logger: gettimeofday needs <sys/time.h>

Either works on linux, but kfreebsd build fails
if we don't use the <sys/time.h> include.

According to man gettimeofday the correct include is <sys/time.h>

Signed-off-by: Andreas Henriksson <andreas@fatal.se>

build-sys: build libmount everywhere

This enables libmount to build on non-linux, which also
means we can build fsck on non-linux again.
(Since the context part of libmount still needs porting,
building the mount utility has instead been restricted
to only build on Linux.)

This has been build-tested on Debian GNU/kFreeBSD.

Signed-off-by: Andreas Henriksson <andreas@fatal.se>

libmount: only include context on linux

This is part of an attempt to make libmount buildable on non-linux.

The parts that need architecture specific porting is under
the context*.c files and the rest of libmount is useful/used
by for example fsck.

Signed-off-by: Andreas Henriksson <andreas@fatal.se>

libmount: only invoke loopcxt on linux

This is part of an attempt to make libmount buildable on non-linux.

The support for /dev/loop* is Linux-specific so just disable
it on non-linux for now.

Signed-off-by: Andreas Henriksson <andreas@fatal.se>

umount: read mountinfo if --detach-loop given

Don't use the optimization not to read mountinfo from commit
6a52473ecd877227f6f7da2b95da0b51593ffec1 if --detach-loop was given
since we need the name of the loop device in that case and with the
optimization this is not present and thus the detach operation
obviously fails.

Signed-off-by: Robert Schiele <rschiele@gmail.com>

cal: `color` option documentation should match actual behaviour

renice: reorder usage() option descriptions

Make the Usage: and Options: sections to be in same order, which I found
to be quicker to use than alphabetical order.

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

rename: add getpriority() message lookup table

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

renice: fix numeric uid argument parsing

The following was inconflict with what usage() tells are valid option
arguments.

$ renice 1 -u 1000
renice: unknown user 1000
$ id
uid=1000(kerolasa) ...

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

renice: avoid having same lines of code twice

Add getprio() function to avoid duplication of a simple task.

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

renice: disallow --priority <arg> without pid argument

Earlier a lonely priority with an argument but without pid resulted to no
action and success, when the invocation should have failed.

$ renice --priority 42 ; echo $?
0

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

rename: use usage and version print out macros

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

renice: reorder functions to avoid need of function prototype

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

textual: share crypt() error message in sulogin and newgrp

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

newgrp: avoid use of obsolete getpass() function

Read a password from user with termios, and once the password data is no
longer needed ensure it gets overwrote before unallocating memory.

Reference: http://man7.org/linux/man-pages/man3/getpass.3.html
Reference: https://www.securecoding.cert.org/confluence/display/seccode/MSC06-C.+Beware+of+compiler+optimizations
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

newgrp: use libc function to read gshadow if it is available

The glib versionf of getsgnam() is using /etc/nsswitch.conf, allowing the
group passwords to come from external database.

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

setarch: use personality() system call when it is available

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

setarch: reindent code

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

hwclock: remove referal to deprecated keyboard interface

The KDGHWCLK has been gone quite a while.

Reference: http://marc.info/?l=linux-kernel&m=104171103925897
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

eject: make open_device() and select_speed() to use struct eject_control

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

eject: add struct eject_control to remove global variables

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

mountpoint: simplify if statement

Returning straight after print_devno() makes the code to be more obvious
and removes need for long else statement.

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

mkfs.minix: fix couple compiler warnings

disk-utils/mkfs.minix.c:366:3: warning: ISO C forbids 'return' with
expression, in function returning void [-Wpedantic]

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

mountpoint: add struct mountpoint_control

This unifies variable names in different functions, and removes redundant
stat() calls.

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