]> granicus.if.org Git - linux-pam/commit
pam_tally2: Remove unnecessary fsync()
authorMaciej S. Szmigiero <mail@maciej.szmigiero.name>
Mon, 25 Feb 2019 19:50:48 +0000 (20:50 +0100)
committerTomáš Mráz <t8m@users.noreply.github.com>
Tue, 26 Feb 2019 09:06:07 +0000 (10:06 +0100)
commitb136bff25e93be6f11de74aca03569022364b973
treea9980cab20bbb0095762e6da23e721fdfc8076f7
parent6fadf6b0ea489abc47003076263fc7465325f788
pam_tally2: Remove unnecessary fsync()

pam_tally2 does fsync() after writing to a tally file.
This causes hard drive cache flushes on every failed SSH login on many
(if not most) filesystems.
And an internet-exposed machine can have a lot of these failed logins.

This operation however doesn't seem to be necessary - the pam_tally2
module does not do any operation which would need explicit post-crash
ordering, it just does simple file reads and writes.
And doing a fsync() after them doesn't close any race if the system happens
to crash between a write being posted and its fsync() completion.

Let's remove this operation to get rid of all these extra cache flushes.
modules/pam_tally2/pam_tally2.c