summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOliver Kiddle <opk@zsh.org>2015-07-31 15:01:30 +0200
committerOliver Kiddle <opk@zsh.org>2015-07-31 15:01:30 +0200
commit3d9a8073c236b4cb2f7a237ca83c133e55d49f49 (patch)
treed9513b099f577eb9316626c9427c4c75a70d569c
parent77a8cd73903b06e67b1f427b042a0a90b0191b84 (diff)
downloadzsh-3d9a8073c236b4cb2f7a237ca83c133e55d49f49.tar.gz
zsh-3d9a8073c236b4cb2f7a237ca83c133e55d49f49.zip
35963: simplify condition found by coverity to include tautology
-rw-r--r--ChangeLog3
-rw-r--r--Src/Modules/system.c3
2 files changed, 4 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index fa46e98ce..c638e5f43 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
2015-07-31 Oliver Kiddle <opk@zsh.org>
+ * 35963: Src/Modules/system.c: simplify condition found by
+ coverity scan to include tautology
+
* Matthew Martin: 35960 (tweaked): Completion/Unix/Command/_find:
complete correct options for OpenBSD's find
diff --git a/Src/Modules/system.c b/Src/Modules/system.c
index a1ed33a96..1ab1fb17e 100644
--- a/Src/Modules/system.c
+++ b/Src/Modules/system.c
@@ -311,8 +311,7 @@ bin_sysopen(char *nam, char **args, Options ops, UNUSED(int func))
int write = OPT_ISSET(ops, 'w');
int append = OPT_ISSET(ops, 'a') ? O_APPEND : 0;
int flags = O_NOCTTY | append | ((append || write) ?
- (read ? O_RDWR : O_WRONLY) :
- (!append || read) ? O_RDONLY : O_WRONLY);
+ (read ? O_RDWR : O_WRONLY) : O_RDONLY);
char *opt, *ptr, *nextopt, *fdvar;
int o, fd, explicit = -1;
mode_t perms = 0666;