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 2014/10/01 22:11:34 UTC

[jira] [Commented] (THRIFT-1248) pointer subtraction in TMemoryBuffer relies on undefined behavior

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

Jens Geyer commented on THRIFT-1248:
------------------------------------

Hi [~fanatoly],

any progress on this?

> pointer subtraction in TMemoryBuffer relies on undefined behavior
> -----------------------------------------------------------------
>
>                 Key: THRIFT-1248
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1248
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Library
>    Affects Versions: 0.5
>            Reporter: Anatoly Fayngelerin
>            Assignee: Anatoly Fayngelerin
>             Fix For: 1.0
>
>         Attachments: 0_7_0_ptrdiff.patch
>
>
> The ensureCanWrite function in TMemoryBuffer currently "rebases" the buffer pointers by subtracting the original buffer pointer from the pointer newly returned by realloc. While this seems to work fine on my linux setup(I couldn't force a reproducer), pointer subtraction between pointers that we allocated separately is an undefined operation.
> I have run into problems with this on platforms other than linux.
> I am attaching a patch that removes the undefined operation.



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