Bug 1562386 Comment 46 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### Beta/Release Uplift Approval Request
* **User impact if declined**: The networkID may be incorrect for users who have multiple active network adapters.
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: The changes just involve sorting the network identifiers before hashing.
Should be low risk as we've manually verified correctness.
* **String changes made/needed**: 

### Beta/Release Uplift Approval Request
* **User impact if declined**: The networkID may be incorrect for users who have multiple active network adapters.
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: The changes just involve sorting the network identifiers before hashing.
Should be low risk as we've manually verified correctness.
* **String changes made/needed**:
### Beta/Release Uplift Approval Request
* **User impact if declined**: The networkID may be incorrect for users who have multiple active network adapters.
* **Is this code covered by automated tests?**: No
* **Has the fix been verified in Nightly?**: Yes
* **Needs manual test from QE?**: No
* **If yes, steps to reproduce**: 
* **List of other uplifts needed**: None
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: The changes just involve sorting the network identifiers before hashing.
Should be low risk as we've manually verified correctness.
* **String changes made/needed**:

Back to Bug 1562386 Comment 46