From 9b519599538f59a475ed8bdc143e3a2938cda1ba Mon Sep 17 00:00:00 2001 From: Erez Zadok Date: Wed, 24 Nov 2010 14:11:04 -0500 Subject: [PATCH] Amd map.c: don't ignore return value from write(2) Signed-off-by: Erez Zadok --- amd/map.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/amd/map.c b/amd/map.c index 0b06f0d9..ff6014c9 100644 --- a/amd/map.c +++ b/amd/map.c @@ -440,13 +440,18 @@ void notify_child(am_node *mp, au_etype au_etype, int au_errno, int au_signal) { amq_sync_umnt rv; + int err; if (mp->am_fd[1] >= 0) { /* we have a child process */ rv.au_etype = au_etype; rv.au_signal = au_signal; rv.au_errno = au_errno; - write(mp->am_fd[1], &rv, sizeof(rv)); + err = write(mp->am_fd[1], &rv, sizeof(rv)); + /* XXX: do something else on err? */ + if (err < sizeof(rv)) + plog(XLOG_INFO, "notify_child: write returned %d instead of %d.", + err, (int) sizeof(rv)); close(mp->am_fd[1]); mp->am_fd[1] = -1; } -- 2.43.0