From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kinglong Mee Subject: Re: [PATCH 1/4] fs_pin: Fix uninitialized value in fs_pin Date: Fri, 08 May 2015 08:36:11 +0800 Message-ID: <554C04FB.10802@gmail.com> References: <554A149B.5060102@gmail.com> <554A14DB.1020504@gmail.com> <20150507194335.GA16527@fieldses.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: Al Viro , linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , NeilBrown , Trond Myklebust , kinglongmee-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org To: "J. Bruce Fields" Return-path: In-Reply-To: <20150507194335.GA16527-uC3wQj2KruNg9hUCZPvPmw@public.gmane.org> Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-fsdevel.vger.kernel.org On 5/8/2015 3:43 AM, J. Bruce Fields wrote: > On Wed, May 06, 2015 at 09:19:23PM +0800, Kinglong Mee wrote: >> Without initialized, done in fs_pin at stack space may >> contains strange value. > > Looks like both init_fs_pin callers use some variation on kzalloc(), so > I don't think there's any actual bug here. Yes, you are right. But nfsd's export cache using kmalloc for memory allocating, if I insert a fs_pin into svc_export{}, the value of done maybe strange value. Maybe I should update using kzalloc in nfsd? But I'd like this one. thanks, Kinglong Mee > > Maybe there's some other reason for the belt-and-suspenders approach, > that's Al's call, I think. > > --b. > >> >> Signed-off-by: Kinglong Mee >> --- >> include/linux/fs_pin.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/include/linux/fs_pin.h b/include/linux/fs_pin.h >> index 3886b3b..18fad53 100644 >> --- a/include/linux/fs_pin.h >> +++ b/include/linux/fs_pin.h >> @@ -16,6 +16,7 @@ static inline void init_fs_pin(struct fs_pin *p, void (*kill)(struct fs_pin *)) >> INIT_HLIST_NODE(&p->s_list); >> INIT_HLIST_NODE(&p->m_list); >> p->kill = kill; >> + p->done = 0; >> } >> >> void pin_remove(struct fs_pin *); >> -- >> 2.4.0 >> > -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html