MAAS

Merge lp://qastaging/~lamont/maas/bug-1655049 into lp://qastaging/~maas-committers/maas/trunk

  1. bug-1655049
  2. Merge into trunk
Proposed by LaMont Jones
Status: Merged
Approved by: LaMont Jones
Approved revision: no longer in the source branch.
Merged at revision: 5627
Proposed branch: lp://qastaging/~lamont/maas/bug-1655049
Merge into: lp://qastaging/~maas-committers/maas/trunk
Diff against target: 35 lines (+16/-1)
2 files modified
src/maasserver/utils/mac.py (+3/-1)
src/maasserver/utils/tests/test_mac.py (+13/-0)
To merge this branch: bzr merge lp://qastaging/~lamont/maas/bug-1655049
Reviewer Review Type Date Requested Status
Mike Pontillo (community) Approve
Review via email: mp+314348@code.qastaging.launchpad.net

Commit message

Protect get_vendor_for_mac() against IndexError.

Description of the change

Bug#1628761 can also result in IndexError being raised by netaddr.EUI().oui.registration().

To post a comment you must log in.
Revision history for this message
Mike Pontillo (mpontillo) wrote :

This is unfortunate, but it looks good. Thanks for the fix.

We should probably raise this upstream.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.