Re: unused fs/jffs2/acl.c:jffs2_clear_acl()

David Woodhouse wrote:
> On Thu, 2006-06-29 at 15:01 +0200, Adrian Bunk wrote:
> 
>>it might not have been intended that jffs2_clear_acl() in Linus' tree
>>is unused?
> 
> 
> I suspect you're right -- thanks for pointing it out.
> 
> Kaigai-san?

I'm sorry, it's a serious BUG.
When an inode is cleared, jffs2_clear_acl() should be called
to release on-memory ACL. Because the current implementation
didn't care about this cleaning-up, we have memory-leaking.

Please wait a patch for a while.

Thanks,
-- 
KaiGai Kohei <[email protected]>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • unused fs/jffs2/acl.c:jffs2_clear_acl()
    • From: Adrian Bunk <[email protected]>
  • Re: unused fs/jffs2/acl.c:jffs2_clear_acl()
    • From: David Woodhouse <[email protected]>

• Prev by Date: Re: [PATCH] 2.6.17-rt1 : fix x86_64 oops
• Next by Date: ACX100 (softmac-based) driver ready to merge, but is it legal? -- Re: wireless (was Re: 2.6.18 -mm merge plans)
• Previous by thread: Re: unused fs/jffs2/acl.c:jffs2_clear_acl()
• Next by thread: [PATCH-v2] Documentation: remove duplicated words
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]