[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Libguestfs] [PATCH] Fix errno check in readdir in devsparts.c



On Thu, Aug 06, 2009 at 02:40:52PM +0100, Matthew Booth wrote:
> ---
>  daemon/devsparts.c |    8 +++++---
>  1 files changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/daemon/devsparts.c b/daemon/devsparts.c
> index b89682c..e9c5e8f 100644
> --- a/daemon/devsparts.c
> +++ b/daemon/devsparts.c
> @@ -40,7 +40,6 @@ foreach_block_device (block_dev_func_t func)
>    int size = 0, alloc = 0;
>  
>    DIR *dir;
> -  struct dirent *d;
>    int err = 0;
>  
>    dir = opendir ("/sys/block");
> @@ -49,8 +48,11 @@ foreach_block_device (block_dev_func_t func)
>      return NULL;
>    }
>  
> -  errno = 0;
> -  while ((d = readdir (dir)) != NULL) {
> +  while(1) {
> +    errno = 0;
> +    struct dirent *d = readdir(dir);
> +    if(NULL == d) break;
> +
>      if (strncmp (d->d_name, "sd", 2) == 0 ||
>          strncmp (d->d_name, "hd", 2) == 0 ||
>          strncmp (d->d_name, "vd", 2) == 0 ||
> -- 
> 1.6.2.5

ACK.

Rich.

-- 
Richard Jones, Emerging Technologies, Red Hat  http://et.redhat.com/~rjones
virt-top is 'top' for virtual machines.  Tiny program with many
powerful monitoring features, net stats, disk stats, logging, etc.
http://et.redhat.com/~rjones/virt-top


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]