Support new oui.txt file format

Bug #1463842 reported by Jean-Michel Vourgère
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OCS Inventory: Server
New
Undecided
Unassigned
ocsinventory-server (Debian)
Fix Released
Unknown

Bug Description

Hi

I fixed this Debian bug request about not using an extra copy of oui.txt inside ocsinventory, but use the system one:
https://bugs.debian.org/481301

First, I was unable to find any real use of that file:
It's read in /require/function_ipdiscover.php in function loadMac(), and so far as I can tell only called from /plugins/main_sections/ms_ipdiscover/ms_ipdiscover.php. But I was unable to find any real use. So maybe this file should just go away?

The main problem was that oui.txt file format changed and now contains extra spaces at the begining of each line. This is trivial to fix (see patch). I tested the result by appending "var_dump($_SESSION['OCS']["mac"])" at the end of /plugins/main_sections/ms_ipdiscover/ms_ipdiscover.php.
You might want to commit it so you don't have problem next time you refresh oui.txt from upstream.

Full commit is available at:
https://anonscm.debian.org/cgit/collab-maint/ocsinventory-server.git/commit/?id=19c4f871bb96809911120617a060f493346f07f7

Enjoy

--
Nirgal

Revision history for this message
Jean-Michel Vourgère (jmv-deb) wrote :
Changed in ocsinventory-server (Debian):
status: Unknown → Fix Committed
Changed in ocsinventory-server (Debian):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.