Author: metze Date: 2006-07-26 12:25:16 +0000 (Wed, 26 Jul 2006) New Revision: 17256
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=17256 Log: fix 2 crash bugs, which are introduced by making parameters ref pointers! I'm sure there're more places and more care is needed when idl files are changed. Hopefully testing against windows in the build farm find such bugs in future... Why is in the client library this no more possible: NTSTATUS foo([in,out,ref] uint8 *foo); and then just r.in.foo = &foo; status = dcerpc_foo(p, mem_ctx, &r); and r.out.foo will set to r.in.foo via pidl magic, that worked some time ago... metze Modified: branches/SAMBA_4_0/source/torture/rpc/wkssvc.c Changeset: Modified: branches/SAMBA_4_0/source/torture/rpc/wkssvc.c =================================================================== --- branches/SAMBA_4_0/source/torture/rpc/wkssvc.c 2006-07-26 11:41:08 UTC (rev 17255) +++ branches/SAMBA_4_0/source/torture/rpc/wkssvc.c 2006-07-26 12:25:16 UTC (rev 17256) @@ -30,11 +30,13 @@ { NTSTATUS status; struct wkssvc_NetWkstaGetInfo r; + union wkssvc_NetWkstaInfo info; uint16_t levels[] = {100, 101, 102, 502}; int i; BOOL ret = True; r.in.server_name = dcerpc_server_name(p); + r.out.info = &info; for (i=0;i<ARRAY_SIZE(levels);i++) { r.in.level = levels[i]; @@ -71,6 +73,7 @@ r.in.ctr = &ctr; r.in.max_buffer = (uint32_t)-1; r.in.resume_handle = &resume_handle; + r.out.ctr = &ctr; r.out.resume_handle = &resume_handle; printf("testing NetWkstaTransportEnum\n");
