diff options
author | Daniel Friesel <derf@derf.homelinux.org> | 2009-11-20 20:46:54 +0100 |
---|---|---|
committer | Daniel Friesel <derf@derf.homelinux.org> | 2009-11-20 20:47:02 +0100 |
commit | 41a6aa36f216e3020c51643c0dc8fce926d67b98 (patch) | |
tree | 2169de705fdc89a52add6996ae70680640abcfa9 | |
parent | bd732f8f882ab1c5699ccbd8f1618995a46e5890 (diff) |
store_open: Update the return NULL comment
-rw-r--r-- | src/envstore.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/envstore.c b/src/envstore.c index 5ca1eec..cf4930b 100644 --- a/src/envstore.c +++ b/src/envstore.c @@ -27,10 +27,9 @@ static FILE * store_open(char *file) { uid_t self_uid = geteuid(); FILE *fp = fopen(file, "r"); - /* Assume the store file does not exist and the store is empty - * (-> return NULL for empty list) - * If it does in fact exist but has wrong permissions or similar, - * the user will only notice if it happens with envstore save/rm (FIXME) + /* + * err() should be used here, but since envstore save may or may not have + * an existing store to work with, return NULL if there's no usable store. */ if (fp == NULL) return NULL; |