Bug #613

pncSaveHostAP() does not properly identify the interface name or write it's IP based address.

Added by Hammel over 2 years ago. Updated over 1 year ago.

Status:ClosedStart date:20 Mar 2018
Priority:ImmediateDue date:
Assignee:Hammel% Done:

100%

Category:-
Target version:PiBox - 1.1.0 - Upgrades
Severity:03 - Medium

Description

While testing Ironman pair-mode I found that pncSaveHostAP() is not properly identifying the requested interface for the AP. It also isn't properly building the AP IP from the base address.

Associated revisions

Revision 08ea51e2
Added by Hammel over 2 years ago

RM #613: Fix identification of requested interface. Fix building IP address for from base address.

Revision 08ea51e2
Added by Hammel over 2 years ago

RM #613: Fix identification of requested interface. Fix building IP address for from base address.

Revision a814bd56
Added by Hammel over 2 years ago

RM #613: When there is no interface for host ap (as there isn't when setting up AP in Ironman monitor node) then create one first before trying to save it's configuration.

Revision a814bd56
Added by Hammel over 2 years ago

RM #613: When there is no interface for host ap (as there isn't when setting up AP in Ironman monitor node) then create one first before trying to save it's configuration.

Revision 87fd9704
Added by Hammel over 2 years ago

RM #613: Typo fix: change strup to strdup.

Revision 87fd9704
Added by Hammel over 2 years ago

RM #613: Typo fix: change strup to strdup.

History

#1 Updated by Hammel over 2 years ago

  • % Done changed from 0 to 30

There were some basic bugs in pncSaveHostAP() that I've fixed. But now I'm up against a new one: the code tries to update an existing interface entry. Problem is, there is no entry for the uap0 interface in /etc/network/interfaces. The fix is to add a new entry if no match is found.

There is some work required to integrate that so I'm making this note so I can do it later.

#2 Updated by Hammel over 2 years ago

  • Severity changed from 01 - Critical to 03 - Medium

#3 Updated by Hammel over 1 year ago

  • Target version changed from 1.0 - Atreides to 1.1.0 - Upgrades

#4 Updated by Hammel over 1 year ago

  • Status changed from In Progress to Closed
  • % Done changed from 30 to 100

It as apparent looking at the associated commits that the last problem - adding an interface if it doesn't exist - has already been implemented. Considering that the problem was uncovered with Ironman and that system has been working well for some time I think it's safe to close this issue.

Closing issue.

Also available in: Atom PDF