]> git.ipfire.org Git - thirdparty/asterisk.git/commit
main/utils: Don't emit an ERROR message if the read end of a pipe closes 10/1810/3
authorMatt Jordan <mjordan@digium.com>
Sun, 13 Dec 2015 19:13:55 +0000 (13:13 -0600)
committerMatt Jordan <mjordan@digium.com>
Sun, 13 Dec 2015 19:22:52 +0000 (13:22 -0600)
commitd4e349129345d35c33dc3312f6b58497dde39a28
tree962acd3d1a07f389d32260500f510e5b322bde37
parent62574fb9dcbecf3dcd3b83b619900656ea1bd3db
main/utils: Don't emit an ERROR message if the read end of a pipe closes

An ERROR or WARNING message should generally indicate that something has gone
wrong in Asterisk. In the case of writing to a file descriptor, Asterisk is not
in control of when the far end closes its reading on a file descriptor. If the
far end does close the file descriptor in an unclean fashion, this isn't a bug
or error in Asterisk, particularly when the situation can be gracefully
handled in Asterisk.

Currently, when this happens, a user would see the following somewhat cryptic
ERROR message:

  "utils.c: write() returned error: Broken pipe"

There's a few problems with this:
(1) It doesn't provide any context, other than 'something broke a pipe'
(2) As noted, it isn't actually an error in Asterisk
(3) It can get rather spammy if the thing breaking the pipe occurs often, such
    as a FastAGI server
(4) Spammy ERROR messages make Asterisk appear to be having issues, or can even
    mask legitimate issues

This patch changes ast_carefulwrite to only log an ERROR if we actually had one
that was reasonably under our control. For debugging purposes, we still emit
a debug message if we detect that the far side has stopped reading.

Change-Id: Ia503bb1efcec685fa6f3017bedf98061f8e1b566
main/utils.c