On Fri, Sep 3, 2010 at 3:52 PM, Ivana Hutarova Varekova
<[email protected]> wrote:
> This patch adds the missing test to cgroup_read_stats_end
>
> Signed-off-by: Ivana Hutarova Varekova <[email protected]>
> ---
>
>  src/api.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/src/api.c b/src/api.c
> index 492fb51..e56776c 100644
> --- a/src/api.c
> +++ b/src/api.c
> @@ -2876,6 +2876,9 @@ int cgroup_read_stats_end(void **handle)
>                return ECGINVAL;
>
>        fp = (FILE *)*handle;
> +       if (fp == NULL)
> +               return 0;
> +

no, it should return an error. handle shuold never be null.

>        fclose(fp);
>        return 0;
>  }
>
>
> ------------------------------------------------------------------------------
> This SF.net Dev2Dev email is sponsored by:
>
> Show off your parallel programming skills.
> Enter the Intel(R) Threading Challenge 2010.
> http://p.sf.net/sfu/intel-thread-sfd
> _______________________________________________
> Libcg-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/libcg-devel
>

------------------------------------------------------------------------------
This SF.net Dev2Dev email is sponsored by:

Show off your parallel programming skills.
Enter the Intel(R) Threading Challenge 2010.
http://p.sf.net/sfu/intel-thread-sfd
_______________________________________________
Libcg-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/libcg-devel

Reply via email to