r23661: Another static pstring
authorVolker Lendecke <vlendec@samba.org>
Fri, 29 Jun 2007 17:40:37 +0000 (17:40 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 17:23:43 +0000 (12:23 -0500)
(This used to be commit d4256ae5588fddc62bf90082d24140e327a54c8c)

source3/rpc_server/srv_ntsvcs_nt.c

index 79259174fd2d376965fed0029e676dd7c617ac8f..6164217a65f3ea6e2d7de516aec207f47196a5c5 100644 (file)
 /********************************************************************
 ********************************************************************/
 
-static char* get_device_path( const char *device )
+static char* get_device_path(TALLOC_CTX *mem_ctx, const char *device )
 {
-       static pstring path;
-
-       pstr_sprintf( path, "ROOT\\Legacy_%s\\0000", device );
-
-       return path;
+       return talloc_asprintf(mem_ctx, "ROOT\\Legacy_%s\\0000", device);
 }
 
 /********************************************************************
@@ -52,16 +48,21 @@ WERROR _ntsvcs_get_version( pipes_struct *p, NTSVCS_Q_GET_VERSION *q_u, NTSVCS_R
 WERROR _ntsvcs_get_device_list_size( pipes_struct *p, NTSVCS_Q_GET_DEVICE_LIST_SIZE *q_u, NTSVCS_R_GET_DEVICE_LIST_SIZE *r_u )
 {
        fstring device;
-       const char *devicepath;
+       char *devicepath;
 
        if ( !q_u->devicename )
                return WERR_ACCESS_DENIED;
 
        rpcstr_pull(device, q_u->devicename->buffer, sizeof(device), q_u->devicename->uni_str_len*2, 0);
-       devicepath = get_device_path( device );
+
+       if (!(devicepath = get_device_path(p->mem_ctx, device))) {
+               return WERR_NOMEM;
+       }
 
        r_u->size = strlen(devicepath) + 2;
 
+       TALLOC_FREE(devicepath);
+
        return WERR_OK;
 }
 
@@ -72,17 +73,21 @@ WERROR _ntsvcs_get_device_list_size( pipes_struct *p, NTSVCS_Q_GET_DEVICE_LIST_S
 WERROR _ntsvcs_get_device_list( pipes_struct *p, NTSVCS_Q_GET_DEVICE_LIST *q_u, NTSVCS_R_GET_DEVICE_LIST *r_u )
 {
        fstring device;
-       const char *devicepath;
+       char *devicepath;
 
        if ( !q_u->devicename )
                return WERR_ACCESS_DENIED;
 
        rpcstr_pull(device, q_u->devicename->buffer, sizeof(device), q_u->devicename->uni_str_len*2, 0);
-       devicepath = get_device_path( device );
+
+       if (!(devicepath = get_device_path(p->mem_ctx, device))) {
+               return WERR_NOMEM;
+       }
 
        /* This has to be DOUBLE NULL terminated */
 
        init_unistr2( &r_u->devicepath, devicepath, UNI_STR_DBLTERMINATE );
+       TALLOC_FREE(devicepath);
        r_u->needed = r_u->devicepath.uni_str_len;
 
        return WERR_OK;