Github user DaveBirdsall commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1723#discussion_r225662018
  
    --- Diff: core/sql/executor/ExExeUtilGet.cpp ---
    @@ -1665,10 +1665,10 @@ NABoolean 
ExExeUtilGetMetadataInfoTcb::checkUserPrivs(
       if (ComUser::isRootUserID())
         return FALSE;
     
    -  // any user granted the DB__ROOTROLE sees everything
       Int32 numRoles;
       Int32 *roleList;
    -  if (currContext->getRoleList(numRoles, roleList) == SUCCESS)
    +  Int32 *granteeList;
    --- End diff --
    
    Who deletes the memory for the granteeList? (or the roleList for that 
matter)


---

Reply via email to