kv_flat_btree_async.cc: release AioCompletion before leave the loop

CID 727982 (#1 of 1): Resource leak (RESOURCE_LEAK)
  leaked_storage: Variable "aioc" going out of scope leaks the storage
  it points to.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
mds/Migrator.cc: fix possible NULL pointer dereference

Move dout() calls behind the related asserts to prevent possible NULL
pointer dereference.

CID 717001 (#1 of 1): Dereference null return value (NULL_RETURNS)
  dereference: Dereferencing a pointer that might be null "diri" when calling
  "operator <<(std::ostream &, CInode

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
mds/Migrator.cc: fix dereference NULL return value

CID 717000 (#1 of 2): Dereference null return value (NULL_RETURNS)
  dereference: Dereferencing a pointer that might be null "dir" when
  calling "operator <<(std::ostream &, CDir &)".

CID 717000 (#2 of 2): Dereference null return value (NULL_RETURNS)
  dereference: Dereferencing a pointer that might be null "dir" when
  calling "Migrator::import_reverse_unfreeze(CDir *)".

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
mds/Migrator.cc: fix possible dereference NULL return value

CID 716999 (#1 of 1): Dereference null return value (NULL_RETURNS)
  dereference: Dereferencing a pointer that might be null "in" when
  calling "CInode::put_stickydirs()".

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
mds/Migrator.cc: delete some empty lines at EOF

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
mds/Migrator.cc: fix possible dereference NULL return value

ID 716998 (#1 of 2): Dereference null return value (NULL_RETURNS)
  dereference: Dereferencing a pointer that might be null "in" when
  calling "operator <<(std::ostream &, CInode &)".

CID 716998 (#2 of 2): Dereference null return value (NULL_RETURNS)
  dereference: Dereferencing a pointer that might be null "in" when
  calling "MDCache::add_replica_dir(ceph::buffer::list::iterator &,
  CInode *, int, std::list<Context *, std::allocator<Context *> > &)".

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
mds/Migrator.cc: fix possible dereference NULL return value

CID 716997 (#1 of 1): Dereference null return value (NULL_RETURNS)
  dereference: Dereferencing a pointer that might be null "in" when
  calling "MDSCacheObject::is_auth() const".

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Merge pull request #310 from dalgaaf/wip-da-CID-fixes-4

Reviewed-by: Sage Weil <sage@inktank.com>
Merge pull request #297 from dalgaaf/wip-da-CID-727982

Reviewed-by: Sage Weil <sage@inktank.com>
ceph: remove cli test

This is about to be removed by wip-ceph-cli anyway.  And it broke in
commit 132d5bf7f9af7de9e2028e20c95ba91637da5875.

Signed-off-by: Sage Weil <sage@inktank.com>
common: add mask argument to pick_addresses() to specify what we need

Fixes: #4228

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
ceph-mon: only care about public addr during pick_addresses()

Fixes: #4228

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
ceph-osd: specify which types of addresses to pick

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
mon: PaxosService: drop atomic_t on 'proposing'

We don't need this to be atomic -- a simple boolean is enough.

Fixes: #4507

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
Merge pull request #315 from ceph/wip-4507

Reviewed-by: Sage Weil <sage@inktank.com>
ceph-mds: specify which types of addresses to pick

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
ceph-syn: specify which types of addresses to pick

Signed-off-by: Joao Eduardo Luis <joao.luis@inktank.com>
Merge pull request #314 from ceph/wip-4228

Reviewed-by: Sage Weil <sage@inktank.com>