Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.Sign up
[dev.icinga.com #1879] idomod: change stacked memory allocation for broker_data IDO_MAX_BUFLEN #733
This issue has been migrated from Redmine: https://dev.icinga.com/issues/1879
Created by mfriedrich on 2011-09-08 14:38:24 +00:00
there's the problem with the default buffer on stack allocation with the change of IDO_MAX_BUFLEN from 16K to 64K in combination with a horrible mklivestatus client thread implementation, causing write_to_all_logs invoking a threaded neb callback to idomod broker_log_data, calling up a stacked temp_buffer with 50KB, while mklivestatus thread size is only 64 KB.
the allocation of memory for that buffer needs to be changed, not the size as is.
further information below (german only)
2011-09-08 14:40:34 +00:00 by mfriedrich 4ccb136
Updated by mfriedrich on 2011-09-22 18:29:32 +00:00
this was resolved during a 6h debug session and works fine now. the livestatus git holds a fix for themselves which wouldn't cause client threads to use non-threadsafe core functionality.