]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
XioMessenger.cc: fix dead pointer usage
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Tue, 3 Feb 2015 21:34:34 +0000 (22:34 +0100)
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Tue, 3 Feb 2015 21:51:47 +0000 (22:51 +0100)
Move _addr to upper scope to prevent dead pointer 'a' after
assignment of _addr in if-clause.

Fix for:

[src/msg/xio/XioMessenger.cc:669]: (error) Dead pointer usage.
Pointer 'a' is dead if it has been assigned '&_addr' at line 644

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
src/msg/xio/XioMessenger.cc

index 1b9ea7c312276b12699a24c7c52d45243ecc587d..44ab40ba98ba193030d9da4adedf7ac8d085b770 100644 (file)
@@ -639,8 +639,9 @@ xio_place_buffers(buffer::list& bl, XioMsg *xmsg, struct xio_msg*& req,
 int XioMessenger::bind(const entity_addr_t& addr)
 {
   const entity_addr_t *a = &addr;
+  struct entity_addr_t _addr = *a;
+
   if (a->is_blank_ip()) {
-    struct entity_addr_t _addr = *a;
     a = &_addr;
     std::vector <std::string> my_sections;
     g_conf->get_my_sections(my_sections);