You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@thrift.apache.org by "pengzhouhu (Jira)" <ji...@apache.org> on 2019/09/06 09:20:00 UTC

[jira] [Updated] (THRIFT-4952) Modified infinite loop in ssl_read

     [ https://issues.apache.org/jira/browse/THRIFT-4952?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

pengzhouhu updated THRIFT-4952:
-------------------------------
    Description: 
   when we call this SSL_read fuction  "bytes = SSL_read(ssl_socket->ssl, buf, len) ", bytes defined  guint , if SSL_read fuction returns a negative number, it will be converted to a large int number, which will break all the time.

Considering the read field size of SSL，so i think change type from 'guint' to 'gint32' is enough.



  was:
   when we call this SSL_read fuction  "bytes = SSL_read(ssl_socket->ssl, buf, len) ", bytes defined  guint , if SSL_read fuction returns a negative number, it will be converted to a large int number, which will break all the time as a infinite loop.

Considering the read field size of SSL，so i think change type from 'guint' to 'gint32' is enough.




> Modified infinite loop in ssl_read
> ----------------------------------
>
>                 Key: THRIFT-4952
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4952
>             Project: Thrift
>          Issue Type: Bug
>    Affects Versions: 0.13.0
>         Environment: thrift-master
>            Reporter: pengzhouhu
>            Priority: Major
>             Fix For: 0.13.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
>    when we call this SSL_read fuction  "bytes = SSL_read(ssl_socket->ssl, buf, len) ", bytes defined  guint , if SSL_read fuction returns a negative number, it will be converted to a large int number, which will break all the time.
> Considering the read field size of SSL，so i think change type from 'guint' to 'gint32' is enough.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)