[PATCHSET] Several fixes found by our tools

Andrew Bartlett abartlet at samba.org
Wed May 16 18:56:22 UTC 2018


On Wed, 2018-05-16 at 19:20 +0200, Andreas Schneider via samba-
technical wrote:
> Hi,
> 
> I've packaged Samba for the next RHEL version and our internal tools found 
> several issues. Attached is a patchset which mostly fixes complaints by 
> Coverity.
> 
> 
> Please review and push if OK.
> 

I've reviewed and pushed:
s3:smbd: Fix converity warning with _smb_setlen_large()
libcli: Fix coverity warning in smb2cli_notify_send()
s4:torture: Make sure variable is initialized in oplock test
s3:modules: Initialize pointers in vfs_virusfilter
s3:winbind: Initialize validation_level in winbind_dual_SamLogon()
s4:dsdb:tests: Add return code check
ctdb: Check return values of tevent_req_set_endtime()

Others I couldn't review before breakfast:

* I don't like the close(-1) patches, that seems harmless really. 
Someone else may however feel it is better to shut up the tool.

* The printf log patch I looked reversed and otherwise strange, so I've
left it for brighter eyes to check. 

Thanks!

Andrew Bartlett

-- 
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba




More information about the samba-technical mailing list