* Dhaval Giani <[email protected]> [2010-01-04 18:04:42]:

> Since we add new errors after ECGSENTINEL, it makes no
> sense to have the assert in place. Remove the assert.
> 
> Signed-off-by: Dhaval Giani <[email protected]>
> 
> ---
>  src/api.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Index: libcg/src/api.c
> ===================================================================
> --- libcg.orig/src/api.c
> +++ libcg/src/api.c
> @@ -106,6 +106,7 @@ char *cgroup_strerror_codes[] = {
>       "Cgroup, rules file does not exist",
>       "Cgroup mounting failed",
>       "The config file can not be opened",
> +     "Sentinel"
>       "End of File or iterator",
>  };
> 
> @@ -2496,7 +2497,6 @@ cleanup_path:
> 
>  char *cgroup_strerror(int code)
>  {
> -     assert((code >= ECGROUPNOTCOMPILED) && (code < ECGSENTINEL));
>       if (code == ECGOTHER) {
>               return strerror_r(cgroup_get_last_errno(), errtext, MAXLEN);
>       }
>

Hmm... I should have caught it before, given we've let it slip, the
patch makes sense. 

-- 
        Balbir

------------------------------------------------------------------------------
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
_______________________________________________
Libcg-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/libcg-devel

Reply via email to