diff options
| author | Alasdair G Kergon <agk@redhat.com> | 2016-06-23 21:31:49 +0100 |
|---|---|---|
| committer | Alasdair G Kergon <agk@redhat.com> | 2016-06-23 21:31:49 +0100 |
| commit | e99a31c9502a2081c6608a831b601081008cc3b0 (patch) | |
| tree | 801442ba7a27872d19aaabd5b2b9e2d53079f4be /lib | |
| parent | a67a5d465576d4a7c17b22a737725d6dee059e18 (diff) | |
| download | lvm2-e99a31c9502a2081c6608a831b601081008cc3b0.tar.gz | |
Revert "locking: trace errors from dir creation"
This reverts commit fa69ed0bc845df3d2c7ae68d03cdd4a3dec339d8.
This code sometimes expects to be presented with a read-only filesystem
(during some boot sequences for example) and copes appropriately with
this and it should not lead to expected error messages that might cause
unnecessary alarm.
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/locking/file_locking.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/lib/locking/file_locking.c b/lib/locking/file_locking.c index 390f49ffe..230303b47 100644 --- a/lib/locking/file_locking.c +++ b/lib/locking/file_locking.c @@ -158,13 +158,11 @@ int init_file_locking(struct locking_type *locking, struct cmd_context *cmd, (void) dm_prepare_selinux_context(NULL, 0); if (!r) - return_0; + return 0; /* Trap a read-only file system */ - if ((access(_lock_dir, R_OK | W_OK | X_OK) == -1) && (errno == EROFS)) { - log_sys_error("access", _lock_dir); + if ((access(_lock_dir, R_OK | W_OK | X_OK) == -1) && (errno == EROFS)) return 0; - } return 1; } |