You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Jens Geyer (Jira)" <ji...@apache.org> on 2019/10/17 22:29:00 UTC
[jira] [Commented] (THRIFT-4952) Modified ssl_read feedback value
break all the time error.
[ https://issues.apache.org/jira/browse/THRIFT-4952?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16954146#comment-16954146 ]
Jens Geyer commented on THRIFT-4952:
------------------------------------
Good catch!
> Modified ssl_read feedback value break all the time error.
> ----------------------------------------------------------
>
> Key: THRIFT-4952
> URL: https://issues.apache.org/jira/browse/THRIFT-4952
> Project: Thrift
> Issue Type: Bug
> Components: C glib - Library
> Affects Versions: 0.13.0
> Environment: thrift-master
> Reporter: pengzhouhu
> Assignee: pengzhouhu
> Priority: Major
> Fix For: 0.14.0
>
> Time Spent: 20m
> 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.4#803005)