Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ngx_http_vhost_traffic_status_module.c #4

Closed
wants to merge 1 commit into from

Conversation

itpp16
Copy link

@itpp16 itpp16 commented Mar 8, 2015

\nginx-module-vts\src\ngx_http_vhost_traffic_status_module.c(395) : warning C4700: uninitialized local variable 'vtsn' used
\nginx-module-vts\src\ngx_http_vhost_traffic_status_module.c(395) : warning C4700: uninitialized local variable 'ms' used
ms fix is ok, not sure if vtsn should be zero to start with.

\nginx-module-vts\src\ngx_http_vhost_traffic_status_module.c(395) : warning C4700: uninitialized local variable 'vtsn' used
\nginx-module-vts\src\ngx_http_vhost_traffic_status_module.c(395) : warning C4700: uninitialized local variable 'ms' used
ms fix is ok, not sure if vtsn should be zero to start with.
@vozlt
Copy link
Owner

vozlt commented Mar 9, 2015

Thanks for the patch!
It applied with some fix.
Is the compile okay without warning?

7d87876

@itpp16
Copy link
Author

itpp16 commented Mar 9, 2015

That works! tnx!
I did find one weird thing, when it starts you see the uptime counting up, 30s..31s.. all the way to 59s and then it continues 60..61..62.. to 99s and then it displays 1m 0s and counts time properly. So the first minute up-time is 99seconds :)

@vozlt
Copy link
Owner

vozlt commented Mar 9, 2015

Thank you for the report. I fixed it.
08e1705

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants