On Friday 2022-12-02 08:13, Al Viro wrote: >On Fri, Dec 02, 2022 at 03:03:31PM +0800, Li Qiong wrote: >> The 'ret' should need to be initialized to 0, in case >> return a uninitialized value. > >Why is 0 the right value? And which case would it be? >We clearly need to know that to figure out which return >value would be correct for it... Judging from the error-handling branches, it should be ret = NF_ACCEPT.