Hallo,
ich habe grad eine dhcp Unterstützung in eins meiner Projekte
implementiert und bin dabei über einen, meiner Meinung nach, Fehler on
Michaels Implementierung des DHCP-Clients gestoßen.
Diese Implementierung ist mir des öfteren über den Weg gelaufen, weshalb
ich denke, dass sie sehr verbreitet ist.
1 | volatile unsigned long dhcp_lease;
|
Nach einer erfolgreichen Adresszuteilung gibt die Debug die Lease Time
aus, die zunächst noch im Cache (Byte-Array) steht:
1 | DHCP_DEBUG("LEASE %2x%2x%2x%2x\r\n", cache.lease[0],cache.lease[1],cache.lease[2],cache.lease[3]);
|
(Hier ist ein kleiner Fehler, der nicht schlimm, aber unschön ist, denn
meiner Meinung nach müsste es so heißen:
1 | DHCP_DEBUG("LEASE %02x%02x%02x%02x\r\n", cache.lease[0],cache.lease[1],cache.lease[2],cache.lease[3]);
|
Als nächstes wird die Zeit in einem unsigned long mit Hilfe eines casts
gespeichert:
1 | dhcp_lease = *((unsigned long*)&cache.lease[0]);
|
Und hier liegt meiner Meinung nach der Fehler. Wenn man die Byte-Order
eines unsigned long im avr-gcc betrachtet, so wird die Zeit falsch
gesetzt. Unter zur Hilfenahme der vorhandenen Funktionen / Makros müsste
es doch
1 | dhcp_lease = HTONS(*((unsigned long*)&cache.lease[0]));
|
heißen, oder?!
dhcp_lease wird durch den Timer sekündlich dekrementiert. Der Fehler
sorgt imho dafür, dass das Device seine Zuteilung nicht rechtzeitig
verlängert. Dies kann dann nach einer gewissen Zeit, nämlich wenn der
Server die IP an einen anderen client vergibt, für Probleme sorgen!
Wenn ich falsch liege, bitte klärt mich auf.
Fabian
