tests: cal: take account week numbers when determing month lenght

The month header does not need to wrap if month width with week number
allows printing to single line.

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

cal: determine output width at beginning of run and reuse result

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

cal: simplify ascii_weeknum() function

Either the week is or isn't highlighted, there is no need to over
complicate printing of that.

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

cal: make day_in_week() use same variable names as other functions

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

cal: add input variable names to function prototypes

While prototypes such as 'function(int, int, char, int);' are legal they
do not help when trying to understand how the function is expected to be
used.  Adding variable names gives at least a hint when looking the
prototypes.

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

cal: rename ascii_wnum() to ascii_weeknum()

Week is not long word, so it does not need to be enshorted and made more
difficult to understand.

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

cal: use control structure for run time configuration

Runtime configuration is set at the time of command line option parsing,
and other initialization.  Later the configuration is read-only.  This
should make code a little bit more understandable.

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

script: script input redirection / eof handling

  echo "ps uf" | script

does not work because script assume that stdin is terminal and
it does not forward EOF to the pty.

This patch:

  * make non-tty use-case more robust (don't call tty ioclts to
    non-tty file descriptors.

  * send EOF (CTL('D') control char) to the master channel when
    detected eof by read() on stdin

  * wait for empty master and slave file descriptors to be sure that
    we don't miss date for typescript. This is also necessary to be
    sure that slave channel (shell) is completely initialized otherwise
    EOF is ignored.

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

hexdump: Create struct hexdump containing previously global variables.

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

hexdump: add __attribute__ ((__noreturn__)) to bad*() functions

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

hexdump: rename struct _fs to struct hexdump_fs and remove its typedef

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

hexdump: rename struct _fu to struct hexdump_fu and remove its typedef

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

hexdump: Rename struct _pr to struct hexdump_pr and remove it's typedef

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

hexdump: rename rewrite to rewrite_rules

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

hexdump: rename add to add_fmt

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

hexdump: Merge hexsyntax.c into hexdump.c

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

hexdump: add the prefix 'hexdump-' to {conv,display,parse}.c

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

script: check that stdin is a terminal

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

setterm: fix -dump man page info

It seems that setterm -dump does not dump terminal attributes, but
the man page says -dump includes the attributes. Let's fix the man
page.

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

agetty: support the special terminal on first serial line on a S/390

which is due legacy reasons a block terminal of type 3270 or higher.
Whereas the second serial line on a S/390(x) is a real character
terminal which is compatible with VT220.

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

mount man page cleanups

Fowarding this patch submitted by Bjarni Ingi Gislason
<bjarniig@rhi.hi.is> to deb bug #674595.

mount.8: fix grammar

Switch "A unbindable" to "An unbindable"

Reported-by: Regid Ichira <regid23@nt1.in>
Signed-off-by: Phillip Susi <psusi@ubuntu.com>

pylibmount: correctly import from pylibmount.so

Without this, python is unable to find the module:

$ python -c 'import libmount'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/lib/python3.3/site-packages/libmount/__init__.py", line 1, in <module>
    from pylibmount import *
ImportError: No module named 'pylibmount'

Signed-off-by: Dave Reisner <dreisner@archlinux.org>

namei: align missing files with rest of the print out

This change should make namei output more readable when multiple
arguments are supplied.

The old:

$ namei -l /usr/foo
f: /usr/foo
drwxr-xr-x root root /
drwxr-xr-x root root usr
foo - No such file or directory

The new:

$ namei -l /usr/foo
f: /usr/foo
drwxr-xr-x root root /
drwxr-xr-x root root usr
                     foo - No such file or directory

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

losetup, agetty: remove unnecessary if's before free()

Reference: http://git.savannah.gnu.org/cgit/gnulib.git/tree/build-aux/useless-if-before-free
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

libfdisk: (dos) warn on type 0

This is old warning, but somehow lost during rewrite to libfdisk. Sorry.

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

libfdisk: (dos) fix free part counter

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

libblkid: fix sgi PT parser to report system partitions

... to be compatible with Linux kernel.

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

fdisk: (gpt) more human readable info about free space

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

fdisk: don't care about partition type

fdisk allows to overwrite existing partition if the partition has
partition type set to zero (very unusual, but possible).

This is incorrect because such partition is pretty valid (and Linux
kernel does not care about the type at all). For example:

   Device Boot      Start         End      Blocks   Id  System
/dev/sdb1            2048       22527       10240   83  Linux
/dev/sdb2           22528      227327      102400    0  Empty
                    ^^^^^                            ^^^^^^^^

Command (m for help): n
Partition type:
   p   primary (1 primary, 0 extended, 3 free)
   e   extended
Select (default p):
Using default response p
Partition number (3,4, default 3):
First sector (22528-1228799, default 22528):
                                    ^^^^^^^

.. if user follows the default then the result are overlapping
partitions:

   Device Boot      Start         End      Blocks   Id  System
/dev/sdb1            2048       22527       10240   83  Linux
/dev/sdb2           22528      227327      102400    0  Empty
/dev/sdb3           22528     1228799      603136   83  Linux
                    ^^^^^

... and if you call "mkfs /dev/sdb3" then you lost data on sdb2. Sad
thing.

Reported-by: Marc MERLIN <marc@merlins.org>
Signed-off-by: Karel Zak <kzak@redhat.com>

fdisk: fix nasty coding style

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

fdisk: fix usage(), -l [<disk>]

Add square brackets, <disk> is optional for -l.

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

umount: fix umount by tag for non-roots

 $ umount LABEL=foo

currently does not work for non-root users as umount uses the
LABEL= as a path in sanitize_path().

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

libblkid: fix memory leak in blkid_parse_tag_string()

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

mount: make NAME=value tags usable for non-root

The libmount does not care if we set source or target, it's able to
swap it, but the mount.c function sanitize_paths() does not work as
expected if we set NAME=value as target. It means that

 $ mount LABEL=foo

does not work for non-root users (since 51e3530cdcb1d4f3ab91ae953ebc5adcdc5f9239, v2.24).

This patch also checks if source or target is specified more
than once.

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

libmount: add mnt_tag_is_valid()

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

fsck: add ext4 to list of filesystems with progress bars in fsck man page

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

mount: fix man mount page type

Slight grammatical error in the mount man page

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

setpriv: simplify usage()

Let's keep usage() simple and details in the man page. The current
brief description in usage() seems confusing for some users.

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

Merge branch 'fixes' of https://github.com/rudimeier/util-linux

* 'fixes' of https://github.com/rudimeier/util-linux:
  tests: fix fdisk GPT test

libfdisk: (sgi) generate partname according to partition position

The SGI drive currently counts only usable partitions, that's wrong as
Linux kernel counts all partition to generate partition number. For
example:

Kernel:
sdb                                             8:16   0   600M  0 disk
├─sdb9                                          8:25   0     2M  0 part
└─sdb11                                         8:27   0 596.2M  0 part

Fdisk:
Pt# Device    Info Start       End   Sectors Id System
 9: /dev/sdb1          0      4095      4096  0 SGI volhdr
11: /dev/sdb2          0   1220939   1220940  6 SGI volume

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

unshare: add more hints about mount namespaces to the man page

There is also idea (by Lennart) to add --propagation=[shared|private|slave|off]
to unshare(1), but it seems unnecessary and too complex as everyone can call
mount(8) after unshare(1) to setup a proper namespace.

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

lsblk: fix -D segfault

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

mkfs.minix: fix fscanf() format string [coverity scan]

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

tests: fix fdisk GPT test

There was a missing newline since cd05de50.

Signed-off-by: Ruediger Meier <ruediger.meier@ga-group.nl>

script: restore errno in signal handler

References: https://plus.google.com/+LennartPoetteringTheOneAndOnly/posts/gHSscCJkakd
Signed-off-by: Karel Zak <kzak@redhat.com>

script: don't call TIOCGWINSZ in signal handler

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

mkswap: remove cruft from the man page

I don't think there's any reason to continue to mention behavior
of kernels older than 10 years.

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

mount: correct mount man page default iso9660 permission

The mount man page said the default permissions are read for
everyone, but it is in fact, read and execute.

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

libmount: cleanup fix_optstr() regards to selinux and smack

It's better (more robust) to make the code based on datetypes
rather than on if(strcmp(...)).

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

build-sys: add --with-smack to config-gen.d

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

libmount: fix typo in smack path

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

libmount: remove smackfs* option when SMACK not enabled

If there is no /sys/fs/smackfs then libmount removes smackfs*= mount
options when compiled --with-smack.

Note that we do the same for SELinux.

References: http://www.mail-archive.com/systemd-devel@lists.freedesktop.org/msg13740.html
Signed-off-by: Karel Zak <kzak@redhat.com>

sulogin: use dirent->d_type when scans /dev

... it's more effective than call fstatat() for all device.

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

docs: update links to http://git.kernel.org/ web repository views

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

docs: add patching process to howto-contribute.txt

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

chsh, chfn: add info about non-local support to the man pages

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

tests: update blkid swap tests

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

hexdump: catch memory leaks

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

hexdump: make addfile() variable names more hinting of their purpose

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

hexdump: use skip_space() in add()

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

strutils: add skip_space() function

[kzak@redhat.com: - add also skip_blank(),
                  - remove duplicate implementation from libmount]

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

hexdump: rename in() to first_letter()

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

hexdump: get rid of redundant typecasts

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

hexdump: minor formatting improvements in display()

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

hexdump: rewrite cluttered case statements

[kzak@redhat.com: - use strchr() for in() macro]

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

hexdump: use xasprintf in conv_c()

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

hexdump: delete redundant typecasts

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

hexdump: use GNU case ranges where possible

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

hexdump: use xstrncpy in add()

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

hexdump: simplify newsyntax arguments

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

hexdump: minor formatting improvements

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

hexdump: rename next{fs,fu,pr} to {fs,fu,pr}list

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

hexdump: rewrite addfile() to use getline()

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

hexdump: formatting and variable name cleanup

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

hexdump: rewrite() formatting improvements

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

hexdump: rewrite escape()

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

hexdump: rewrite rewrite()

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

hexdump:rewrite add()

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

hexdump: rewrite next()

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

hexdump: rewrite get()

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

hexdump: rewrite addfile()

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

hexdump: cleanup of redundant symbols/repeating literals

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

hexdump: use list.h queues and rewrite redundant for cycles

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

hexdump: add basic tests

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

blkid: simple typo

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

docs: add note about make distcheck

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

blkid: escape quotes in the output

 # e2label /dev/loop0 'La"bel'
 # blkid -p /dev/loop0
 /dev/loop0: LABEL="La"bel"  ....

new version:

 /dev/loop0: LABEL="La\"bel" ....

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

tests: add sparc64 lscpu test

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

lscpu: support non sequentially numbering of CPUs

lscpu don't work correctly on my system with:

 $ cat /sys/devices/system/cpu/possible
 0-1,4-5,8-9,12-13

[kzak@redhat.com: - coding style,
                  - add commit message
                  - add real_cpu_num() macro,
                  - fix functions where we need idx as well as CPU number]

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

blockdev: don't use HDIO_GETGEO

blockdev was still using this depreciated ioctl and that was
causing blockdev --report to fail on loop and nbd devices.
Switch to reading the partition start from sysfs instead.
This also allows it to correctly report > 2^32 sector counts.

[kzak@redhat.com: - check sysfs_init() return,
                  - use uint64_t rather than unsigned long long]

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

blkdiscard: BLKSSZGET fills in an int, not a uint64

Reported-by: Jason Cipriani <jason.cipriani@gmail.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>

libmount: update utab after successful extern helper mount

This patch allows to maintain private utab libmount file also for
external mount/umount helpers that are not linked with libmount.

The libmount check if utab has been updated after successful extern
helper execution (status=0). If not then the file is updated.

This patch affects only 'user' fstab mount option. So, for example
with suid mount.cifs you can use:

 //server/foo    /mnt  cifs   username=foo,noauto,user

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

libmount: allow to use -s (sloppy) for all external helpers

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

libblkid: fix swap-area version

 * v1: hdr->version = 1, magic string = SWAPSPACE2
 * v0: no versio in header, magic string = SWAP-SPACE

The mistake has been introduced in the old udev volume_id
binary where the version was set according to magic string.

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

libfdisk: (sun) fix end sectors with +1 error

partx (or parted, etc.):

NR     START       END   SECTORS   SIZE NAME UUID
 1         0    410047    410048 200.2M
 2    410048  82329055  81919008  39.1G
 4  82329056 102807615  20478560   9.8G
 5 102807616 110999039   8191424   3.9G
 6 110999040 312580543 201581504  96.1G

and the same with fdisk:

     Device Flag    Start       End    Blocks   Id  System
/dev/loop0p1             0    410048    205024    1  Boot
/dev/loop0p2        410048  82329056  40959504   83  Linux native
/dev/loop0p3             0 312580544 156290272    5  Whole disk
/dev/loop0p4      82329056 102807616  10239280   83  Linux native
/dev/loop0p5     102807616 110999040   4095712   83  Linux native
/dev/loop0p6     110999040 312580544 100790752   8e  Linux LVM

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

mount: improve -s man mage info

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

libfdisk: (gpt) add /home GUID

Reported-by: andreas.hanke@gmx-topmail.de
References: http://cgit.freedesktop.org/systemd/systemd/commit/?id=1a14a53cfded6e78c6e8dfb73fdff0039971d642
Signed-off-by: Karel Zak <kzak@redhat.com>

libfdisk: (gpt) allow to work with hybrid GPT

Notes:

 * fdisk don't sync hybrid MBR with GPT
 * hybrid MBR is ignored and not overwritten by PBMR
 * users is informed about expert command 'M' to toggle between MBR
   and GPT mode. It's possible to manually modify hybrid MBR.

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

libfdisk: (gpt) improve and cleanup recovery code

 * use AlternativeLBA rather than hardcoded offset to backup GPT
 * create gpt_copy_header()

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