11 Commits (mr13.4)

Author	SHA1	Message	Date
Richard Fuchs	56eddd464c	MT#62181 XmlRpcClient: fix incorrect printf format ... Change-Id: If205d38cf55435fe8d0ac2f2ab800a5d845a4c5c Warned-by: Coverity	1 year ago
Donat Zenichev	9a74726171	MT#59962 XmlRpcSocket: refactor `nbWrite()`, make it safer ... Rework the `nbWrite()` function so, that it gets safe in terms of: - operating on data written/data remaining integers - has protection guards for extra big chunks to write at once Refactor to use size_t/ssize_t instead of primitive int, where possible (e.g. bytesWritten). Let the `nbWrite()` use reference instead of pointer for the bytes written parameter. Fixes Coverity Scan report: 4. tainted_data_return: Called function write(fd, sp, nToWrite), and a possible return value may be less than zero. 5. cast_overflow: An assign that casts to a different type, which might trigger an overflow. Overflowed integer argument (INTEGER_OVERFLOW) 13. overflow_sink: nToWrite, which might have overflowed, is passed to write(fd, sp, nToWrite). 7. overflow: The expression nToWrite is considered to have possibly overflowed. Change-Id: I32aa6aae5ef5715d61a91714e62b0a094bc03f21	1 year ago
Donat Zenichev	7a46e8e09c	MT#59962 xmlrpc2di: refactor `XmlRpcClient::close()` ... Don't try to double free the `_ssl_ssl` and `_ssl_ctx` provided by `XmlRpcSource`, because this job is already done by `XmlRpcSource::close()`. Additionally: - add a separate non-null check for the _ssl_ctx freeing - move log lines from `XmlRpcClient::close()` to `XmlRpcSource::close()` Change-Id: Ifef799fa6a0ffa65b60415f39c51f4b54ff4bb14	1 year ago
Donat Zenichev	0a5b30dc96	MT#59962 xmlrpc2di: check XmlRpcSource existence before using ... Don't appeal to the XmlRpcSource's members (e.g. `_ssl`) if this has been freed via `this`. To check this out see controllable delete behavior via introduced getter `getsDeletedOnClose()`. Should fix: freed_arg: close frees this. CID 542402: (#1 of 1): Read from pointer after free (USE_AFTER_FREE) use_after_free: Using freed pointer this. Change-Id: I8f23564957c57491db8cf051e7296671bf2a698a	1 year ago
Richard Fuchs	1b695faabb	MT#62181 fix initialisers ... Add missing initialisers and fix order. In some cases the member was actually unused and could just be removed. Change-Id: I0f0c927eb8271c35dcfd371f225847f62bea2812 Warned-by: Coverity	1 year ago
Richard Fuchs	b357698a75	MT#62181 use size_t for offsets and lengths ... Change-Id: I8a4630ad1d7fae351f96cd479f28839724903582	1 year ago
Richard Fuchs	7267f879e0	MT#62181 XmlRpc*: fix printf argument type ... Use a static cast to make sure we use the right type for std::string::length() Change-Id: I3b21146ca8898d7e1f3b35c83389d37eadac5ff6 Warned-by: Coverity	1 year ago
Richard Fuchs	6dbeb9281a	MT#62181 XmlRpcClient: add setFault() ... Make it more explicit that this is meant to set the flag instead of doing a comparison with a typo. Change-Id: I31a1a7f35b9072f157105f6c2acdd2caf9bea09a Warned-by: Coverity	1 year ago
Stefan Sayer	f947af4102	fix some size printf fmt	16 years ago
Stefan Sayer	6d1d1713ef	made condition clearer to the compiler ... git-svn-id: http://svn.berlios.de/svnroot/repos/sems/trunk@1642 8eb893ce-cfd4-0310-b710-fb5ebe64c474	17 years ago
Stefan Sayer	ac2cf0a414	added xmlrpc++ library source, as xmlrpc++ is not any more in debian testing and unfortunately had gentoo eol as well. ... This version is from xmlrpc++_0.7+cvs20040713.tar.gz the debian etch source package http://ftp.de.debian.org/debian/pool/main/x/xmlrpc++/xmlrpc++_0.7+cvs20040713.orig.tar.gz + xmlrpcpp07_sems.patch which just removes some compiler warnings and adds fPIC git-svn-id: http://svn.berlios.de/svnroot/repos/sems/trunk@1210 8eb893ce-cfd4-0310-b710-fb5ebe64c474	18 years ago