Allocating memory to nested Structures within a Structure

Dear Friends

I have the follwoing function "tss_fe_get_own_info" which has the
arguments as shows below
 tss_fe_get_own_info(char *user_id, tss_user_profile_t
**p_buf_UserSecInfo, error_status_t *retStatus)

// Structure tss_user_profile_t from the function

typedef struct {
tss_user_info_t user_info;
odss_attribute_value_list_t *ProcAttr;
odss_attribute_value_list_t *BkOffAttr;
odss_attribute_value_list_t *WkClusterAttr;
odss_attribute_value_list_t *CSUAttr;
} tss_user_profile_t;

// Structure tss_user_info_t
typedef struct
{
  char user_uuid[USER_UUID_LEN+1];
  char user_sec_info_status[USER_ACC_STATUS_LEN+1];
  char user_id[USER_ID_LEN+1];
  char user_name[USER_NAME_LEN+1];
  char employee_id[EMPLOYEE_ID_LEN+1];
  char job_title[JOB_TITLE_LEN+1];
  char job_type[JOB_TYPE_LEN+1];
  char loc_addr[LOC_ADDR_LEN+1];
  char loc_country_code[LOC_CTRY_CODE_LEN+1];
  char loc_city_code[LOC_CITY_CODE_LEN+1];
  char telephone_number[TELE_NUM_LEN+1];
  char expense_code[EXPENSE_CODE_LEN+1];
  char proc_locn_csu[PROC_LOCN_CSU_LEN+1];
  char proc_locn_rpf[RPF_CODE_LEN+1];
  char prim_wk_cluster[PRM_WK_CLUS_LEN+1];
  char experience[EXPERIENCE_LEN+1];
  char oracle_id[ORACLE_ID_LEN+1];
  char oracle_pwd[ORACLE_PWD_LEN+1];
  char user_group[USER_GROUP_LEN+1];
  char date_modified[DATE_STR_LEN+1];
  char date_valid[DATE_STR_LEN+1];
} tss_user_info_t;

// odss_attribute_value_list_t
typedef struct odss_attribute_value_list_t
{
  [ptr] odss_attribute_value_t *attribute_value;
  [ptr] struct odss_attribute_value_list_t *next;
} odss_attribute_value_list_t;

// Structure odss_attribute_value_t

typedef struct
{
  unsigned32 length;
  [string, ptr] char *attribute;
} odss_attribute_value_t;

If I want to allocate the memory to nested structures in that case
I have done the follwoing

tss_user_profile_t **p_buf_UserSecInfo;
*p_buf_UserSecInfo = new tss_user_profile_t;
 memset((void*) *p_buf_UserSecInfo, '\0',
sizeof(tss_user_profile_t)); // at this point the profile is
allocated, as well as the user_info

unsigned32 len1 = 0;
char *attr1 = NULL;

odss_attribute_value_list_t *tmp1 = (*p_buf_UserSecInfo)->ProcAttr; //
Allocates the ProcAttr pointer :

ProcAttr = new odss_attribute_value_list_t;
ProcAttr->attribute_value = new odss_attribute_value_t;
ProcAttr->attribute_value->length = len1;
ProcAttr->attribute_value->attribute = attr1;
ProcAttr->next = tmp1;

unsigned32 len2 = 0;
char *attr2 = NULL;

odss_attribute_value_list_t *tmp2 = (*p_buf_UserSecInfo)-


BkOffAttr = new odss_attribute_value_list_t;
BkOffAttr->attribute_value = new odss_attribute_value_t;
BkOffAttr->attribute_value->length = len2;
BkOffAttr->attribute_value->attribute = attr2;
BkOffAttr->next = tmp2;

unsigned32 len3 = 0;
char *attr3 = NULL;

odss_attribute_value_list_t *tmp3 = (*p_buf_UserSecInfo)-


WkClusterAttr = new odss_attribute_value_list_t;
WkClusterAttr->attribute_value = new odss_attribute_value_t;
WkClusterAttr->attribute_value->length = len3;
WkClusterAttr->attribute_value->attribute = attr3;
WkClusterAttr->next = tmp3;

unsigned32 len4 = 0;
char *attr4 = NULL;

odss_attribute_value_list_t *tmp4 = (*p_buf_UserSecInfo)->CSUAttr; //
Allocates the CSUAttr pointer :

CSUAttr = new odss_attribute_value_list_t;
CSUAttr->attribute_value = new odss_attribute_value_t;
CSUAttr->attribute_value->length = len4;
CSUAttr->attribute_value->attribute = attr4;
CSUAttr->next = tmp4;

Please let me know if this is the correct way to allocate the memory

thanks
Amit