s4:torture: Fix cleanup of the secrets object in session_key test
authorAndreas Schneider <asn@samba.org>
Thu, 17 Nov 2016 15:15:54 +0000 (16:15 +0100)
committerJeremy Allison <jra@samba.org>
Thu, 17 Nov 2016 21:30:36 +0000 (22:30 +0100)
commit9de04626c058563a6cf4c13e4f5399039e345ef5
treeaa322c87d8083aea060f44b9506bf9856c3e177f
parent31d21de548d899f82fa7944767ad94e8aca8d96d
s4:torture: Fix cleanup of the secrets object in session_key test

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12433

The test is known to be failing if sealing is turned on in some
circumstances. In this case a secret is created and then the function
dcerpc_fetch_session_key() fails. The secret is not removed!

We use torturesecret-%08x with random() to fill in the number. Sometimes
it happens that random() returns a number we already used. So we end up
trying to create a secret for an entry which already exists and run
into a collision

This change makes sure we always cleanup behind us and do not leave
secret objects we created.

Pair-Programmed-With: Guenther Deschner <gd@samba.org>
Signed-off-by: Andreas Schneider <asn@samba.org>
Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Thu Nov 17 22:30:36 CET 2016 on sn-devel-144
source4/torture/rpc/session_key.c