On Fri, 9 May 2014 15:00:07 +0300, Dan Carpenter wrote:
> Hello Stefan Behrens,
>
> The patch 5db0276014b8: "Btrfs: add optional integrity check code"
> from Nov 1, 2011, leads to the following static checker warning:
>
> fs/btrfs/check-integrity.c:1099 btrfsic_process_metablock()
> warn: missing error code here? 'btrfsic_stack_frame_alloc()' failed. 'sf->error' = '0'
>
> fs/btrfs/check-integrity.c
> 1092
> 1093 next_stack =
> 1094 btrfsic_stack_frame_alloc();
> 1095 if (NULL == next_stack) {
> 1096 btrfsic_release_block_ctx(
> 1097 &sf->
> 1098 next_block_ctx);
>
> Should we set "sf->error" here? I don't know the code well enough to
> say the answer.
>
> 1099 goto one_stack_frame_backwards;
> 1100 }
> 1101
> 1102 next_stack->i = -1;
Looking at this function immediately made me blind and getting terrible
headache, therefore I can only guess whether not setting sf->error was
intentional or not three years ago. Nowadays, I'd set sf->error since
this propagates the error condition upwards. Although it doesn't make a
difference to the user and doesn't cause a crash whether it is set or
not. But if the function returns an error/success status, and an error
was detected, the error status should be returned. I'll send a patch.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
