mailru / graphite-nginx-module

An nginx module for collecting stats into Graphite
135 stars 34 forks source link

avoid using uninitialized memory #10

Closed arnej27959 closed 8 years ago

arnej27959 commented 8 years ago

we've just tried the graphite-nginx-module and soon saw crashes in template execution. From the core dumps it seems clear that the template compile function relies on zero- initialized data (arg->data.len is never set to 0 for variables). Here's a fix; I can provide more details about how the crash happens if you want.

kirimedia commented 8 years ago

Thanks!