Skip to content

Commit f389e41

Browse files
nioffeGerrit Code Review
authored and
Gerrit Code Review
committed
Merge "Don't SIGABORT if device-mapper doesn't generate uevent for DeleteDevice"
2 parents 3d152c9 + 3b39a9e commit f389e41

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

fs_mgr/libdm/dm.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,10 @@ bool DeviceMapper::DeleteDevice(const std::string& name,
111111

112112
// Check to make sure appropriate uevent is generated so ueventd will
113113
// do the right thing and remove the corresponding device node and symlinks.
114-
CHECK(io.flags & DM_UEVENT_GENERATED_FLAG)
115-
<< "Didn't generate uevent for [" << name << "] removal";
114+
if ((io.flags & DM_UEVENT_GENERATED_FLAG) == 0) {
115+
LOG(ERROR) << "Didn't generate uevent for [" << name << "] removal";
116+
return false;
117+
}
116118

117119
if (timeout_ms <= std::chrono::milliseconds::zero()) {
118120
return true;

0 commit comments

Comments
 (0)