Closed bernardnormier closed 4 weeks ago
This PR updates Ice C++ to make sure the OutputStream held by an OutgoingMessage (see ConnectionI.h) owns its memory.
It's possible to create an OutputStream that doesn't own its memory with: https://github.com/zeroc-ice/ice/blob/e72309cdc7d1f4da178aa6b08d62b7ef0b183820/cpp/include/Ice/OutputStream.h#L65
It would be dangerous to construct an OutgoingResponse (or other outgoing message) with such an OutputStream as it may be sent later on, asynchronously, without copying of the memory.
This PR updates Ice C++ to make sure the OutputStream held by an OutgoingMessage (see ConnectionI.h) owns its memory.
It's possible to create an OutputStream that doesn't own its memory with: https://github.com/zeroc-ice/ice/blob/e72309cdc7d1f4da178aa6b08d62b7ef0b183820/cpp/include/Ice/OutputStream.h#L65
It would be dangerous to construct an OutgoingResponse (or other outgoing message) with such an OutputStream as it may be sent later on, asynchronously, without copying of the memory.