View Issue Details

IDProjectCategoryView StatusLast Update
0000009Freifunk Franken FirmwareGeneralpublic2019-10-02 12:49
ReporterChristianD Assigned Toreddog  
PrioritylowSeveritytweakReproducibilityhave not tried
Status resolvedResolutionfixed 
Platformvermutlich alleOSOpenWRT 
Product Version20161105-beta 
Target Version20170110 
Summary0000009: Zeit scheint falsch zu gehen. NTP Problem?
DescriptionSo wie es aussieht scheint die Uhrzeit falsch zu gehen:

root@THWest:/lib/functions/fff# date
Wed Nov 16 07:52:19 CET 2016

eingegeben am 16.11.2016 um 23:52Uhr.

Eventuell geht auch nur am Netmon NTP Server die Uhrzeit falsch?

Anderseits scheint bei Firmwareversion 20160506 die Uhrzeit richtig zu gehen:

root@Dachaussenrouter:~# date
Wed Nov 16 23:55:01 CET 2016

eingegeben um 23:55Uhr. Vielleicht dann doch ein Problem in der FW?
TagsNo tags attached.

Activities

reddog

2017-01-06 16:12

manager   ~0000032

Hab ein wenig rumgeschaut. Die Validierung selbst ist erstmal nötig, da diese die Variablen in dem Init-Script erst setzt.. Da ist ein eckeliges eval drin. :(

Die Validierung selbst könnte man natürlich fixen, aber dafür muss man in https://git.lede-project.org/?p=project/ubox.git;a=blob;f=validate/validate.c erstmal den String auf '%' untersuchen, diesen dann (bei Bedarf) auseinander schneiden und dann anschließend die zweite Hälfte auch noch auf Gültigkeit prüfen.

Das könnte in etwa so gehen:
---
#include <sys/types.h>
#include <ifaddrs.h>
int main() {
    struct ifaddrs *ifap, *ifa;

    if (getifaddrs(&ifap) != -1)
    {
        for (ifa = ifap; ifa != NULL; ifa = ifa->ifa_next)
        {
            if (ifa->ifa_addr == NULL)
                continue;
            if (ifa->ifa_addr->sa_family != AF_INET6)
                continue;

            printf("%-8s\n", ifa->ifa_name);
        }

        freeifaddrs(ifap);
    }

    return 0;
}
---

Ich glaube aber, dass der Aufwand da jetzt nicht gerechtfertigt ist. Zumal ich vermute, dass das ganze Validieren bald^{TM} mit einem neuen Schema für UCI entfallen könnte.

Mein Ansatz ist daher die IP bei der Validierung einfach als String anzusehen.

reddog

2017-01-10 19:32

manager   ~0000034

Fix committed to master branch.

Issue History

Date Modified Username Field Change
2016-11-16 23:53 ChristianD New Issue
2016-11-16 23:55 ChristianD Description Updated
2016-11-16 23:56 ChristianD Description Updated
2016-12-17 12:17 reddog OS Version 20161105-beta =>
2016-12-17 12:17 reddog Product Version => 20161105-beta
2016-12-17 12:17 reddog Target Version => 20170110
2017-01-06 12:20 mayosemmel Source_changeset_attached => Firmware master ff7e8b7a
2017-01-06 12:24 mayosemmel Assigned To => mayosemmel
2017-01-06 12:24 mayosemmel Status new => resolved
2017-01-06 12:24 mayosemmel Resolution open => fixed
2017-01-06 12:27 reddog Source_changeset_removed Firmware master ff7e8b7a =>
2017-01-06 12:27 reddog Assigned To mayosemmel =>
2017-01-06 12:27 reddog Status resolved => new
2017-01-06 12:27 reddog Resolution fixed => reopened
2017-01-06 16:08 reddog Assigned To => reddog
2017-01-06 16:08 reddog Status new => assigned
2017-01-06 16:12 reddog Note Added: 0000032
2017-01-10 19:32 reddog Source_changeset_attached => Firmware master 303ddf3c
2017-01-10 19:32 reddog Note Added: 0000034
2017-01-10 19:32 reddog Status assigned => resolved
2017-01-10 19:32 reddog Resolution reopened => fixed
2019-10-02 12:48 fbl Category Freifunk Franken Firmware => General
2019-10-02 12:48 fbl Category General => General2
2019-10-02 12:49 fbl Category General2 => General