From 41a6aa36f216e3020c51643c0dc8fce926d67b98 Mon Sep 17 00:00:00 2001 From: Daniel Friesel Date: Fri, 20 Nov 2009 20:46:54 +0100 Subject: store_open: Update the return NULL comment --- src/envstore.c | 7 +++---- 1 file 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; -- cgit v1.2.3