last: tell verbally system is still running

Use of uptime time stamp as previous boot login time makes the output not
constant, which is rather difficult to test.  Verbal message 'system is
still running' makes testing easy, and noticing which boot is still
running clear to a person.

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

last: trim trailing white spaces

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

tests: add last(1) test

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

last: never restrict session time

When a session time will reach whopping 10000 days the last round bracket
is unnecessarily removed from output.

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

docs: add --time-format to last(1) manual page

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

last: add --time-format with iso-8601 format

The ISO-8601 format makes consuming time stamps easy with various
parsers.  The format includes time zone information which is crucial when
an investigator is trying to make sense outputs collected from systems
all a across planet.

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

last: use configuration struct

This allows reducing global variables and will minimize number of
arguments for functions making code a little bit easier to read, and
maintain.

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

last: fix --present option logic error

Printing of sessions 'still logged in' was skipped because an error in
logic.

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

docs: add --since and time option formats to last(1) manual

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

last: add --since time spec option

The --since and --until options are companions often needed together.

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

lib/time-util: move YYYYMMDDHHMMSS to common parser

Even while the YYYYMMDDHHMMSS time format it not magnificent it is best
to make it to be part of the one, and only, time format parser.

Proposed-by: Karel Zak <kzak@redhat.com>
References: http://markmail.org/message/6baqt4ttkopu7ra6
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

last: parse easy to use time formats

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

lib/strutils: move *swith() functions to private library

Avoid code dublication in libmount and time-util.

Proposed-by: Karel Zak <kzak@redhat.com>
Reference: http://markmail.org/message/h7zexvqsieqngtmx
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

lib/time-util: copy time parsing functions from systemd

The functions are copied nearly as-is.  Coding style has been modified to
match with util-linux project, while the functionality remains untouched.

CC: Lennart Poettering <lennart@poettering.net>
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

write: change determination can user write to a terminal

Earlier test failed when user had write permission, but was not in
special group that owns terminal devices, usually tty.

This made write(1) to fail for root, if the root did not happen to be in
tty group.  In this commit root is granted ot write to anyone, even if
they have mesg(1) turned off.  For an user who is trying to write to own
other session the group write bit is significant only for whether mesg(1)
are enabled.

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

docs: add note to wall(1) about sessions which will not get message

Sessions that has in utmp data ut_line begining with ':' are skipped.
This is done to avoid write errors to devices such as '/dev/:0'.  For
example wdm is reported to to generate such utmp line.

Perhaps in future this code should be revalidated.  If an utmp data
contains broken device lines it might be best inform user with a error
message, rather than skipping silently suspicious device entries.

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

docs: add --present to last(1) manual page

I forgot to update manual after writing this functionality.

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

wall: line wrap at column 79 also when line has tab chars

Earlier tabs caused random ragged right indentation, because the tab
length was assumed to be 1 char which is not the case most of the time.

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

mesg: sync usage() with howto-usage-function.txt

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

wall: sync usage() with howto-usage-function.txt

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

last: fix typo

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

wall: send message also to sessions opened by user 'sleeper'

Ignoring an user by name, and not telling about it in manual page, is
unexpected.

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

build-sys: pylibmount will not work when python3 is present

While compiling with up to date Archlinux I notice the recently added
pylibmount does not link correctly.  Failures inform fundamental types
such as PyFileObject and PyFile_Type being missing.

It seems automake AM_PATH_PYTHON will prefer the python in path, which
for this distribution right now is python3.  As some sort of go-around
one can install older python, and symlink it to earlier in PATH lookup.

$ ln -s /usr/bin/python2 $HOME/bin/python
$ export PATH=$HOME/bin:$PATH

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

su: suppress PAM info messages for -c or non-login sessions

The 'Last login:' messages from PAM lastlogin module is unexpected
for non-login sessions or when -c <command> executed.

For example:

  $ su - -c id
  Last login: Wed Jul 24 08:36:28 CEST 2013 from dhcp-25-161.brq.redhat.com on pts/18
  uid=0(root) gid=0(root) skupiny=0(root)

this makes 'su' useless in scripts.

This patch suppress all PAM_TEXT_INFO messages for -c and for
non-login session ('-' is not specified) after pam_authenticate() and
pam_acct_mgmt().

Note that the new PAM conversation function checks the first message
in the msg[] array only. It seems good enough as PAM internally uses
pam_info() function that does not use multiple messages for one conv
call.

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

build-sys: add CFLAGS and LDFLAGS for daemons and shared libs

This is necessary for paranoid security guys who believe that things
like "-Wl,-z,relro" or "-Wl,-z,bind_now" is a way how to make the
world a safer place...

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

tests: minor changes to make tests more portable

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

swapon: cleanup man page

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

swapon: allow a more flexible swap discard policy

Introduce the necessary changes to swapon(8) allowing a sysadmin to leverage
the new changes introduced to sys_swapon by "swap: discard while swapping
only if SWAP_FLAG_DISCARD_PAGES", therefore allowing a more flexible set of
choices when selection the discard policy for mounted swap areas.
This patch introduces the following optional arguments to the already
existent swapon(8) "--discard" option, in order to allow a discard type to
be selected at swapon time:
 * once    : only single-time area discards are issued. (swapon)
 * pages   : discard freed pages before they are reused.
If no policy is selected both discard types are enabled. (default)

[kzak@redhat.com: - support <policy> argument for short -d option too,
                  - add errx() on unsupported policy name]

Signed-off-by: Rafael Aquini <aquini@redhat.com>
Acked-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Signed-off-by: Karel Zak <kzak@redhat.com>

last: mention optional arguments in usage

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

last: add --present option

The --present option will allow user to easily determine who where logged
in at a given time.

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

last: allow --file to be defined multiple times

This is useful when an admin is trying to find something, and has to
process all available data.

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

last: global variables are initialize automatically to zero

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

last: use as narrow variable scoping as possible

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

last: remove broken code

Comment in the deletion tells everything necessary.

"This doesn't work on modern systems, where only a DNS lookup of the
result from hostname() will get you the domainname.  Remember that
domainname() is the NIS domainname, not DNS.  So basically this whole
piece of code is bullshit."

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

last: prefer enum rather than #definition list

Incremental number lists are more hard to get wrong with enum, and they
are nicer to debug as for example gdb is aware of these symbolic names.

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

last: make switch cases complete, and inform if impossible occurs

Some of the ut_type numbers does not seem to be recognized by last(1) so
they are, at least for now, silently ignored.  See glibc documentation
for information what the ignored EMPTY, INIT_PROCESS, LOGIN_PROCESS, and
ACCOUNTING mean.

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

last: check expected numeric user input is number

This commit also changes the line count to use unsigned integers, as
negative numbers in this context does not make sense.

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

last: use carefulput() for printable character output

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

include: carefulput: print determined char when unprintable char is found

This is done to allow reuse of the functin in last(1).

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

docs: add long options to last.1 manual page

And align the manual page with Documentation/howto-man-page.txt

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

docs: be clear which last(1) is deprecated

The recently merged sysvinit last(1) and lastb(1) are supported.

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

build-sys: complete elvtune removal

The first removal managed to remove only most of the elvtune, this commit
will complete the task.

Original-removal: 7f1fe74248290c50f5447c2c09fcf0f0575a7666
Signed-off-by: Sami Kerola <kerolasa@iki.fi>

docs: update TODO

The minix type fixes took place December 2012.

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

build-sys: remove unnecessary environment variable

The addition to original removal done in commit 952f1d79.

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

pylibmount: improve helps strings formatting

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

pylibmount: add debug messages

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

libmount: cleanup libmnt_fs list after mnt_table_remove_fs()

.. otherwise mnt_free_fs() will try to remove FS from non-existing
list.

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

pylibmount: use libmount reference counting

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

misc: use libmnt_table reference counter

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

libmount: add reference counter to libmnt_table

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

misc: use libmnt_cache reference counting

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

libmount: add reference counting to libmnt_cache

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

fsck: use libmount fs reference counter

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

libmount: add reference counting to libmount_fs

 * mnt_new_fs() returns object with refcount=1
 * mnt_free_fs() does not care about reference counter

 * new functions mnt_ref_fs() and mnt_unref_fs()

 * mnt_table_add_fs() and mnt_table_rem_fs() uses reference counter

 * libmmnt_context uses reference counter for internal FS (as it could be
   shared outside the context)

 * backwardly incompatible change:

- FS could be deallocated after mnt_table_remove_fs()

 * it's recommended to use mnt_unref_fs() after mnt_table_add_fs()

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

pylibmount: remove Context_get_table()

It's too problematic and too low-level to support it Py binding.

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

pylibmount: cleanup cxt usage

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

libmount: add function for context userdata

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

pylibmount: remove unnecessary code

It's enough to call mnt_free_fs() to remove fs from the table.

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

libmount: add mnt_table_is_empty(), improve table list usage

Currently you have to use mnt_table_remove_fs() + mnt_free_fs() to
destroy the list in the table. This is complicated in same situations.
This patch allows to use mnt_free_fs() only.

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

pylibmount: coding style changes, don't touch Context() status

It's better to follow libmnt_context status than allow to play any
nasty games with this important variables in Python.

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

pylibmount: use only public libmount API in tab.c

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

libmount: add mnt_table_{first,last}_fs()

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

libmount: add new function to docs

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

libmount: add mnt_table_{set,get}_userdata()

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

libmount: add mnt_table_with_comments()

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

pylibmount: cleanup *_HELP strings formatting

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

pylibmount: use mnt_fs_set_userdata() rather than ->userdata

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

pylibmount: fs.c - cleanup, remove libmount private stuff

 * remove unnecessary comments
 * cleanup up strings format
 * remove direct access to private libmount stuff

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

pylibmount: remove bindsrc from fs API

It's unnecessary and very low-level.

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

pylibmount: cleanup object names

Cxt -> Context
Tab -> Table

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

pylibmount: coding style changes in tab.c

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

pylibmount: coding style changes in fs.c

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

pylibmount: coding style changes in context.c

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

tests: consolidate Py tests initialization

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

build-sys: install pylibmount into

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

pylibmount: add regression tests

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

pylibmount: add __init__.py

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

pylibmount: basic code

[kzak@redhat.com: - split to more patches
                  - split to more .c files]

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

build-sys: add pylibmount

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

build-sys: remove deprecated elvtune command

The command is deprecated for years and useless with 2.6.x kernels.

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

build-sys: enable mesg(1) by default

It seems that our mesg(1) implementation is compatible with sysvinit,
let's use it by default in mainstream distros.

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

wall: make banner sysvinit compatible

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

wall: allow to specify <message> on command line

wall(1) from sysvinit supports

   # wall I love this company!

semantic, This patch add this functionally to the util-linux wall.

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

last: add link to util-linux to last.1

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

last: coding style clean up

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

last: use xalloc.h, err.h, nls.h and EXIT_ macros

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

last: consolidate btmp/wtmp file paths

 .. so share the paths with another utils.

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

last: add long options, improve usage()

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

build-sys: fix last stuff

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

build-sys: improbe UL_CONFLICT_BUILD macro

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

last: clean up the begin of the file

 .. and improve last(1) stuff in configure script.

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

last: merge sysvinit last/lastb

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

agetty: -L accepts optional argument

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

losetup: use loop-control to explicitly ask for device

Now we use LOOP_CTL_GET_FREE ioctl to ask for free device, for example

  losetup -f foo.img

Unfortunately, losetup(8) allows to ask for specified device

  losetup /dev/loop100 foo.img

and in this case we assume that the device already exists in the
system. This is incorrect, we should be able to use loop-control
LOOP_CTL_ADD ioctl to ask for the specified device.

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

blkdiscard: use fstat() rather than stat() [coverity scan]

 Calling function "open(char const *, int, ...)" that uses "path" after
 a check function. This can cause a time-of-check, time-of-use race
 condition.

.. well, in blkdiscard context it's mostly cosmetic change.

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

losetup: fix loop device name usage

The code is not paranoid enough, loopcxt_set_device() only set the
device name to loopdev struct, but it does not check if the device
really exists.

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

libmount: be robust when work with loopdev backing file paths

It's usually unnecessary as we compare devno and ino, but let's
use absolute paths for situations when it's necessary to compare
paths as strings.

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

losetup: make -j <path> more robust

It's usually unnecessary as we compare devno and ino, but let's
use absolute paths for situations when it's necessary to compare
paths as strings.

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

libmount: canonicalize for conversion from loopdev backing file

  # mount foo.img /mnt
  # umount foo.img
  umount: foo.img: not mounted

The loopdev code (and sysfs backing_file) uses absolute paths, but
libmount does not canonicalize the path before lookup for the backing file.

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

cal: set statically defined data read-only

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