Re: [PATCH] cgroup: remove incorrect check on the return value of css_alloc

From: Xiongwei Song
Date: Mon Jan 22 2018 - 18:21:50 EST



> On 22 Jan 2018, at 11:55 PM, Tejun Heo <tj@xxxxxxxxxx> wrote:
>
> Hello,
>
> On Mon, Jan 22, 2018 at 09:38:51PM +0800, Xiongwei Song wrote:
>> The function css_alloc never return NULL, it may return normal pointer or
>
> It's a calling a controller implemented method. I'd much rather keep
> the extra protection.
>
>> error codes that made by ERR_PTR, so !css is always false, we need use
>> IS_ERR to check it, and if this is true, we should use ERR_CAST handle it.
>
> It's of the same type. Why would it need casting?

Okay, Thanks for your comment.

Apologies for the noise.

Thanks
Xiongwei
>
> Thanks.
>
> --
> tejun