Messages in this thread | | | Subject | Re: [PATCH][ocfs2-next] ocfs2: ensure ret is set to zero before returning | From | Joseph Qi <> | Date | Thu, 8 Aug 2019 09:08:46 +0800 |
| |
On 19/8/7 20:42, Colin Ian King wrote: > On 07/08/2019 13:35, Joseph Qi wrote: >> >> >> On 19/8/7 20:19, Colin King wrote: >>> From: Colin Ian King <colin.king@canonical.com> >>> >>> A previous commit introduced a regression where variable ret was >>> originally being set from the return from a call to function >>> dlm_create_debugfs_subroot and this set was removed. Currently >>> ret is now uninitialized if no alloction errors are found which >>> may end up with a bogus check on ret < 0 on the 'leave:' return >>> path. Fix this by setting ret to zero on a successful execution >>> path. >> >> Good catch. >> Or shall we just initialize 'ret' at first? > > Initialized ret first may not catch subsequent coding errors where error > returns paths have not initialized ret, so my preference is when it is > required and not before. > Okay, looks good to me. Reviewed-by: Joseph Qi <joseph.qi@linux.alibaba.com>
>> >>> >>> Addresses-Coverity: ("Uninitialzed scalar variable") > > Can this be fixed up when applied rather sending a V2? >> Currently ocfs2 patches are maintained in Andrew's mm tree. So it depends on Andrew.
Thanks, Joseph
| |