r19451: Another cleanup of 18802. Referring to structures after the block has been

left is not a good idea.

Volker
(This used to be commit 88ac17db196c828e15a73c7561fda291b15e2613)

diff --git a/source3/rpcclient/cmd_srvsvc.c b/source3/rpcclient/cmd_srvsvc.c
index e1e5af752a519f6f1cda8c74e5cbcf7f29f892b4..9f6e606c3a10cffcc59034b5dd9f12a8fcbf2ee0 100644 (file)
--- a/source3/rpcclient/cmd_srvsvc.c
+++ b/source3/rpcclient/cmd_srvsvc.c
@@ -243,6 +243,9 @@ static NTSTATUS cmd_srvsvc_net_share_enum(struct rpc_pipe_client *cli,
                                           int argc, const char **argv)
 {
        uint32 info_level = 2;
+       struct srvsvc_NetShareCtr1 ctr1;
+       struct srvsvc_NetShareCtr2 ctr2;
+       struct srvsvc_NetShareCtr502 ctr502;
        union srvsvc_NetShareCtr ctr;
        NTSTATUS result;
        uint32 hnd;
@@ -263,20 +266,17 @@ static NTSTATUS cmd_srvsvc_net_share_enum(struct rpc_pipe_client *cli,
 
        switch (info_level) {
        case 1: {
-               struct srvsvc_NetShareCtr1 ctr1;
                ZERO_STRUCT(ctr1);
                ctr.ctr1 = &ctr1;
                }
                break;
 
        case 2: {
-               struct srvsvc_NetShareCtr2 ctr2;
                ZERO_STRUCT(ctr2);
                ctr.ctr2 = &ctr2;
                }
                break;
        case 502: {
-               struct srvsvc_NetShareCtr502 ctr502;
                ZERO_STRUCT(ctr502);
                ctr.ctr502 = &ctr502;
                }