lwIP sys_now() function bug

[biafra]

Hi everyone,

It seems I've found in the sys_now() function of the lwIP library:

u32_t sys_now(void)
{
   return xTaskGetTickCount();
}

It returns the number of ticks from startup, not the number of milliseconds!

It should be:

    return xTaskGetTickCount() * portTICK_PERIOD_MS;

Many thanks

Biafra

[danielchen]

Hi Biafra:

Thanks for your attention.

But it seems the LWIP function documentation says, it returns the current time in milliseconds.

Please refer to the below link

lwIP: Time 

Regards

Daniel

[biafra · Accepted solution]

Hi Daniel,

Thanks for yor answer.

That's why I write my post: at the moment by simply calling the xTaskGetTickCount() function the sys_now() function returns the current time in ticks, not in milliseconds. With the correction I suggested it returns milliseconds.

Many thanks

Biafra

[danielchen]

Thanks, Biafra, I will report it to the developers.

Regards

Daniel