And add a test showning the segfault.
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
}
if (lp_obj) {
- bool ok;
+ bool ok = py_check_dcerpc_type(lp_obj, "samba.param",
+ "LoadParm");
+ if (!ok) {
+ PyErr_Format(PyExc_TypeError,
+ "Expected samba.param.LoadParm "
+ "for lp argument");
+ return -1;
+ }
lp_ctx = pytalloc_get_type(lp_obj, struct loadparm_context);
if (lp_ctx == NULL) {
return -1;
assert gpos[i].ds_path == ds_paths[i], \
'ds_path did not match expected %s' % gpos[i].ds_path
+
+ def test_gpo_ads_does_not_segfault(self):
+ try:
+ ads = gpo.ADS_STRUCT(self.server, 42, self.creds)
+ except:
+ pass
+
def test_gpt_version(self):
global gpt_data
local_path = self.lp.get("path", "sysvol")