[PR] INKVConnInternal::do_io_*: handle null buffer [trafficserver]

["bneradt (via GitHub)" <gi...@apache.org>]

bneradt opened a new pull request, #13413:
URL: https://github.com/apache/trafficserver/pull/13413

   It's common for users of VC's to cancel io via a 0 byte, nullptr read or
   write on the VC. INKVConnInternal::do_io_read and
   INKVConnInternal::do_io_write were not prepared to handle such
   cancellations. This updates them to handle this gracefully rather than
   crashing on a nullptr dereference. This change is was found to be needed
   for the multiplexer plugin for handling HttpTunnel aborts.
   
   For reference, see, for example, UnixNetVConnection::do_io_read which
   handles a nullptr MIOBuffer. This basically copies that logic into
   INKVConnInternal so it handles cancellation gracefully.
   
   (cherry picked from commit 7afd9e3dc81229f15d6e9ed1abbb06fc5788182a)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] 9.2.x: INKVConnInternal::do_io_*: handle null buffer [trafficserver]

["ezelkow1 (via GitHub)" <gi...@apache.org>]

ezelkow1 merged PR #13413:
URL: https://github.com/apache/trafficserver/pull/13413


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org