]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
common: Fix multiple logical errors in get_device_id. 30636/head
authorAlex Zhang <difan@google.com>
Sun, 29 Sep 2019 09:33:58 +0000 (02:33 -0700)
committerAlex Zhang <difan@google.com>
Thu, 21 Nov 2019 11:26:26 +0000 (03:26 -0800)
commit3e09dfd8254abf3c4ff85d6c1b1fafc4b38eae7d
tree8507875678cb0000a3e7ad934f4caf1c9d3cc8cf
parente659e860965723899295261bd30a78805639d080
common: Fix multiple logical errors in get_device_id.

0. If blkdev.serial exists, the serial should be used. The original impl seems wrong (if serial does not exist, then use the value from the uninitialized buffer, or even worse, use the value from the last call (model))
1. When using fallback methods, device id should only be returned when both model and serial are present. The original impl looks like a logical error.

Signed-off-by: Difan Zhang <difan@google.com>
src/common/blkdev.cc