Re: [hackers] [sbase][PATCH] ls: abort a directory if we cannot opendir it

From: Hiltjo Posthuma <hiltjo_AT_codemadness.org>
Date: Tue, 22 Aug 2017 13:45:28 +0200

On Tue, Aug 22, 2017 at 04:51:37PM +1200, David Phillips wrote:
> We should not try and perform operations on an invalid DIR* stream.
> Instead, we shall let the error message be printed, and the return
> code set (existing behaviour) and abort afterwards.
> ---
> ls.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/ls.c b/ls.c
> index 5080c8f..b716aba 100644
> --- a/ls.c
> +++ b/ls.c
> _AT_@ -250,6 +250,7 @@ lsdir(const char *path, const struct entry *dir)
> if (!(dp = opendir(dir->name))) {
> ret = 1;
> weprintf("opendir %s%s:", path, dir->name);
> + return;
> }
> if (chdir(dir->name) < 0)
> eprintf("chdir %s:", dir->name);
> --
> 2.14.1
>

Applied, thanks.

-- 
Kind regards,
Hiltjo

Received on Tue Aug 22 2017 - 13:45:28 CEST

This archive was generated by hypermail 2.3.0 : Tue Aug 22 2017 - 13:49:23 CEST