[MESOS-6788] Avoid stack overflow when handling streaming responses in API handlers

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.2.0
• Target Version/s: 1.2.0
• Epic Link: Mesos Container Attach/Exec
• Assignee: Benjamin Hindman
• Reporter: Vinod Kone
• Created: 13/Dec/16 21:55
• Updated: 14/Dec/16 17:44
• Resolved: 14/Dec/16 17:44

Description

Right now both the `connect()` helper in src/slave/http.cpp and `transform()` helper in src/common/recordio.hpp use recursion to read data from one pipe and write to another.

The way these helpers are written could cause stack overflow. Ideally we should be able to leverage the new `process::loop` abstraction for that.