Changeset 8a5de14 in mainline

Timestamp:

2018-03-11T07:35:05Z (6 years ago)

Author:

Jakub Jermar <jakub@…>

Children:

35fd816

Parents:

fb1f664

git-author:

Jakub Jermar <jakub@…> (2018-03-04 17:51:52)

git-committer:

Jakub Jermar <jakub@…> (2018-03-11 07:35:05)

Message:

Handle race condition in phone_connect

Do not panic if the phone is not in the expected state. This could mean
that the user hung up the original phone and created a new one under the
same capability handle.

File:

• kernel/generic/src/ipc/ipcrsc.c (modified) (1 diff)

kernel/generic/src/ipc/ipcrsc.c

@@ -219 +219 @@
                 return false;
 
-        assert(phone_obj->phone->state == IPC_PHONE_CONNECTING);
+        if (phone_obj->phone->state != IPC_PHONE_CONNECTING) {
+                /*
+                 * This looks like another phone. The one we were expecting
+                 * under this handle must be in the IPC_PHONE_CONNECTING state.
+                 */
+                kobject_put(phone_obj);
+                return false;
+        }
 
         /* Hand over phone_obj reference to the answerbox */