[Pidgin] #16327: protobuf-c UB fixes (Gadu protocol)

Pidgin trac at pidgin.im
Wed Aug 27 02:14:01 EDT 2014


#16327: protobuf-c UB fixes (Gadu protocol)
----------------------+--------------------------------------------
 Reporter:  JIghtuse  |      Owner:  EionRobb
     Type:  patch     |     Status:  new
Milestone:            |  Component:  unclassified
  Version:  3.0.0hg   |   Keywords:  UB, Gadu, protobuf, protobuf-c
----------------------+--------------------------------------------
 I run scan-build from the Clang/LLVM project to check Pidgin on build. It
 found 11 problems. Some of them not really a bugs, some are. Full report
 attached as archive scan-build-2014-08-27-1.zip.

 This patch fixes two bugs in protobuf-c, included in Gadu protocol. Both
 of them is undefined behaviour in memcpy(). If we pass NULL pointer as
 source or destination, result of such operation is undefined. GCC-4.9 can
 break some code in such cases  (see
 [https://gcc.gnu.org/gcc-4.9/porting_to.html Null pointer checks may be
 optimized away more aggressively]). Similar changes already included in
 the
 [https://github.com/protobuf-c/protobuf-c/blob/master/protobuf-c/protobuf-c.c
 latest protobuf-c version].

--
Ticket URL: <https://developer.pidgin.im/ticket/16327>
Pidgin <https://pidgin.im>
Pidgin


More information about the Tracker mailing list