| commit | bdb9efef8991396b6c77e669a140965c91bb8558 | |
| author | Tobias Klauser <tklauser@distanz.ch> | |
| Mon, 10 Jun 2013 21:53:28 +0000 (10 23:53 +0200) | ||
| committer | Tobias Klauser <tklauser@distanz.ch> | |
| Mon, 10 Jun 2013 22:02:40 +0000 (11 00:02 +0200) | ||
| tree | d1344b5af39d65846d074dd82185893cddaff030 | treesnapshot (tar.gz zip) |
| parent | 3990796bbb2a122f7f25ca1a399928c9197e5174 | commitdiff |
oui: Make parsing of oui.conf more robust
In the unlikely case of the oui.conf file not conforming to the required
format (e.g. by the user supplying one by himself), we would end up -
in the worst case dereferencing a null pointer, if the condiition
(ptr = strstr(buff, ", ")) is false. Thus make the entire parsing a bit
more robust and just ignore lines not following the required format.
The null pointer dereference was found by the Coverity scanner.
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
In the unlikely case of the oui.conf file not conforming to the required
format (e.g. by the user supplying one by himself), we would end up -
in the worst case dereferencing a null pointer, if the condiition
(ptr = strstr(buff, ", ")) is false. Thus make the entire parsing a bit
more robust and just ignore lines not following the required format.
The null pointer dereference was found by the Coverity scanner.
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
| oui.c | diffblobblamehistory |