[dev.icinga.com #1253] SEGV in ido2db on Solaris with Oracle #561
Comments
Updated by mfriedrich on 2011-03-16 17:44:15 +00:00
|
Updated by mfriedrich on 2011-03-24 22:05:49 +00:00
now it's yours ;-) |
Updated by mfriedrich on 2011-04-27 17:16:57 +00:00
this might need some rework - or is it already done? |
Updated by Tommi on 2011-04-27 19:31:11 +00:00
My patch changes all number types at all, which is wrong approach we learned. Each data type needs to be checked if is large enough in database, sqlstring and coded c data type. My "solution" regarding handling null strings you already rejected. I dont have a new one for now. |
Updated by mfriedrich on 2011-07-16 16:10:57 +00:00
removing release target for proper resolution. |
Updated by Tommi on 2011-07-16 18:14:33 +00:00 i agree. It will take much more time to sort it out. |
Updated by mfriedrich on 2012-04-20 12:42:30 +00:00
|
Updated by mfriedrich on 2014-12-08 14:35:44 +00:00
|
This issue has been migrated from Redmine: https://dev.icinga.com/issues/1253
Created by Tommi on 2011-02-28 20:40:54 +00:00
Assignee: Tommi
Status: Closed (closed on 2012-04-20 12:42:30 +00:00)
Target Version: (none)
Last Update: 2014-12-08 14:35:44 +00:00 (in Redmine)
see http://www.nagios-portal.org/wbb/index.php?page=Thread&threadID=21804
There are 2 problemes
Solution:
1.looks like there is no real usage of unsigned long long in the code, but please check back for measurement values
then replace all OCIBindUnsignetBigInt ((big_uint)...) with OCIBindUnsignetBigInt ((uint)..)
2. Make sure NULL values will be translated into real empty string e.g strdup("") and use functions from supplied snprintf.c in solaris. For this a modification of config.h (better in configure.in) is needed (switch off HAVE_VA_COPY and HAVE_C99_VSNPRINTF), dont use os provided functions like vfprintf which are not covered by snprintf.c. Take care, a syslog call includes such function internally
Attached are diffs for V1.3.0, which are working for me
Attachments
Changesets
2012-01-29 16:17:35 +00:00 by Tommi 06a7dee
2012-02-03 19:42:33 +00:00 by Tommi 543952f
2012-02-03 19:56:08 +00:00 by Tommi a7a9dd5
Relations:
The text was updated successfully, but these errors were encountered: