diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2012-05-03 22:33:54 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2012-05-03 22:34:17 +0100 |
| commit | 76cc8aca4270bc2542cc8444d07f80d578eb6581 (patch) | |
| tree | 3119ba28e5604b62c9d746a77b6110212e31751f /extensions/source/plugin/unx | |
| parent | fff2d18a08a37c0fa6a370e56591117fc08664d2 (diff) | |
WaE: unchecked results from write/fwrite
Change-Id: Ia7943a132d05e596550bd71878b54b2825c31987
Diffstat (limited to 'extensions/source/plugin/unx')
| -rw-r--r-- | extensions/source/plugin/unx/mediator.cxx | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/extensions/source/plugin/unx/mediator.cxx b/extensions/source/plugin/unx/mediator.cxx index a98e19ea71ec..4edefaaab440 100644 --- a/extensions/source/plugin/unx/mediator.cxx +++ b/extensions/source/plugin/unx/mediator.cxx @@ -30,6 +30,7 @@ #include <unistd.h> #include <plugin/unx/mediator.hxx> +#include <sal/log.hxx> #include <vcl/svapp.hxx> #define MEDIATOR_MAGIC 0xf7a8d2f4 @@ -59,7 +60,9 @@ Mediator::~Mediator() aHeader[0] = 0; aHeader[1] = 0; aHeader[2] = MEDIATOR_MAGIC; - write( m_nSocket, aHeader, sizeof( aHeader ) ); + ssize_t nToWrite = sizeof(aHeader); + bool bSuccess = (nToWrite == write(m_nSocket, aHeader, nToWrite)); + SAL_WARN_IF( !bSuccess, "extensions", "short write"); } // kick the thread out of its run method; it deletes itself close( m_nSocket ); @@ -95,7 +98,9 @@ sal_uLong Mediator::SendMessage( sal_uLong nBytes, const char* pBytes, sal_uLong pBuffer[ 1 ] = nBytes; pBuffer[ 2 ] = MEDIATOR_MAGIC; memcpy( &pBuffer[3], pBytes, (size_t)nBytes ); - write( m_nSocket, pBuffer, nBytes + 3*sizeof( sal_uLong ) ); + ssize_t nToWrite = nBytes + 3*sizeof( sal_uLong ); + bool bSuccess = (nToWrite == write( m_nSocket, pBuffer, nToWrite )); + SAL_WARN_IF( !bSuccess, "extensions", "short write"); delete [] pBuffer; return nMessageID; |