[ 
https://issues.apache.org/jira/browse/TRAFODION-2170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15437200#comment-15437200
 ] 

ASF GitHub Bot commented on TRAFODION-2170:
-------------------------------------------

Github user prashanth-vasudev commented on a diff in the pull request:

    https://github.com/apache/incubator-trafodion/pull/672#discussion_r76280881
  
    --- Diff: core/sqf/src/tm/tmddlrequests.cpp ---
    @@ -39,13 +40,24 @@ JNIEXPORT void JNICALL 
Java_org_apache_hadoop_hbase_client_transactional_RMInter
        char la_tbldesc[TM_MAX_DDLREQUEST_STRING];
        char la_tblname[TM_MAX_DDLREQUEST_STRING];
        char* str_key;
    -   str_key = new char[TM_MAX_DDLREQUEST_STRING];
    +   if(pv_keyLength <= 0) 
    +   {
    +     cout << "createTableReq bad input pv_keyLength, abort" << endl;
    +     abort();
    +   }
    +   str_key = new char[pv_keyLength];
    --- End diff --
    
    tmlibmsg.h defines the message types and length. Please check to make sure 
pv_keyLength does not exceed the defined message types. 



> certain create table DDL will fail with core dump
> -------------------------------------------------
>
>                 Key: TRAFODION-2170
>                 URL: https://issues.apache.org/jira/browse/TRAFODION-2170
>             Project: Apache Trafodion
>          Issue Type: Bug
>            Reporter: liu ming
>            Assignee: liu ming
>
>  create table test2170  (
>     c1 varchar(255) CHARACTER SET UTF8 not null, 
>     c2 varchar(256)  CHARACTER SET UTF8 not null, 
>     c3 varchar(256)  CHARACTER SET UTF8 not null, 
>     c4 int not null , 
>      c5 int, 
>     primary key(c1,c2,c3,c4))
> salt using 4 partitions ;
> This will fail.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to