-
Notifications
You must be signed in to change notification settings - Fork 42
Closed
Labels
state:needs-triageNeeds to triaged to determine next stepsNeeds to triaged to determine next stepstype:bugSomething isn't workingSomething isn't workingurgency:mediumImportant with a moderate impactImportant with a moderate impact
Description
Is there an existing issue for this?
- I have searched existing issues
Current Behavior
Registering a callback then using LocateHandle with the search type ByRegisterNotify will result in the callbacks not having access to any of the handles that were present in the core prior to the registration event.
Expected Behavior
EDK II code will return handles to any protocol installed prior to performing the LocateHandle calls.
Steps To Reproduce
Example:
- Protocol 1234 is added to core
- Register for a callback on 1234 with search key so ByRegisterNotify will properly index returned value
- In callback, LocateHandle with search key will report no protocols available
- Add Protocol 1234 again
- In callback, LocateHandle with search key will return just one handle, the second addition of 1234
Running the same experiment on EDK II code will result in step 3 returning a handle to the first installation of protocol 1234.
Build Environment
N/AVersion Information
19.03
Urgency
Medium
Are you going to fix this?
I will fix it
Do you need maintainer feedback?
No maintainer feedback needed
Anything else?
I have a fix already and a PR will be added today.
Metadata
Metadata
Assignees
Labels
state:needs-triageNeeds to triaged to determine next stepsNeeds to triaged to determine next stepstype:bugSomething isn't workingSomething isn't workingurgency:mediumImportant with a moderate impactImportant with a moderate impact