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

    https://github.com/apache/trafodion/pull/1671#discussion_r206455613
  
    --- Diff: core/conn/odb/src/odb.c ---
    @@ -40,6 +40,7 @@ char *odbauth = "Trafodion Dev 
<trafodion-developm...@lists.launchpad.net>";
     #define MAX_ARGS    11          /* Max arguments for interactive mode */
     #define ARG_LENGTH  128         /* Max argument length in interactive mode 
*/
     #define LINE_CHUNK  51200       /* size of memory chunks allocated to 
store lines */
    +#define ERR_MSG_LEN 512         /* size of error message buffer */
    --- End diff --
    
    This value came from the driver implementer @Weixin-Xu 's advise. By the 
way, the small buffer length won't cause the buffer overflow issue, because the 
driver will protect this by truncating the error message.


---

Reply via email to