[PATCH] Fix sigsegv in check_stream in smbtorture smb2.streams.io

Jeremy Allison jra at samba.org
Tue Apr 20 19:55:50 UTC 2021


On Tue, Apr 20, 2021 at 03:36:28PM -0400, Gordon Ross via samba-technical wrote:
>Not 100% sure this is still a defect on master, but it was fairly recently.
>
>If any stream create fails during smb2.streams.io
>the test dies with a sigsegv in torture_comment
>
>torture_comment calls need a struct torture_context arg,
>not its mem_ctx child.  Use talloc_parent().  Also
>need to call torture_result somewhere on failure.

Thanks Gordon ! Probably better to pass down
an extra struct torture_context *tctx parameter
to check_stream() rather than relying on talloc_parent()
to assume it's a tctx.

If I make those changes are you OK with your
"Signed-off-by: Gordon Ross <gordon.ross at tintri.com>"
being added ?

Cheers,

Jeremy.



More information about the samba-technical mailing list