[dev.icinga.com #2234] notifications.cgi - memory access error

[icinga-migration]

This issue has been migrated from Redmine: https://dev.icinga.com/issues/2234

Created by terratectom on 2012-01-02 14:16:27 +00:00

Assignee: ricardo
Status: Resolved (closed on 2012-04-13 15:44:00 +00:00)
Target Version: 1.7
Last Update: 2014-12-08 09:42:27 +00:00 (in Redmine)

Icinga Version: 1.10.0
OS Version: any

---

Calling notifications.cgi breaks for individual days off with memory access error.
Appended report is debug with gdb from the command line.

export REMOTE_USER=...; export REQUEST_METHOD=GET; export QUERY_STRING="archive=4";gdb ./cgi/notifications.cgi

Link to full thred http://www.nagios-portal.org/wbb/index.php?page=Thread&threadID=24720

Attachments

• gdb_output.txt terratectom - 2012-01-02 14:16:27 +00:00
• notifications_segfault_fix.diff ricardo - 2012-01-02 16:11:38 +00:00

Changesets

2012-01-04 22:40:44 +00:00 by ricardo 69a71a0

classic ui: Fixed notifications.cgi - memory access error #2234

refs: #2234

Elements of host and service objects where used
without testing if host/service was found.

[icinga-migration]

Updated by ricardo on 2012-01-02 15:13:02 +00:00

• Status changed from New to Feedback

Hi,

seems that it can't find the service.

did you change your configuration recently?

[icinga-migration]

Updated by ricardo on 2012-01-02 16:11:38 +00:00

• File added notifications_segfault_fix.diff
• Assigned to set to ricardo
• Target Version set to 1.7

Hi,

can you apply these patch to your sources and try if it fixes the issue please?

It seems like you changed the name of the service and this caused trouble. But with this patch the problem should be solved.

[icinga-migration]

Updated by terratectom on 2012-01-03 08:33:40 +00:00

Hi,

this patch have close the bug. Now, all lines issued without error.

[icinga-migration]

Updated by mfriedrich on 2012-01-03 08:35:22 +00:00

• Done % changed from 0 to 90

thanks for the feedback.

@ricardo
pls apply to git for further testing.

[icinga-migration]

Updated by ricardo on 2012-01-04 22:45:01 +00:00

in current "dev/cgis"

[icinga-migration]

Updated by mfriedrich on 2012-04-03 13:51:44 +00:00

• Done % changed from 90 to 100

probably this is a probem with removed already from config hosts in order to attach the notification attributes then. checking if not NULL should be required after invoking find_host() either way. imho this is resolved.

[icinga-migration]

Updated by ricardo on 2012-04-13 15:44:00 +00:00

• Status changed from Feedback to Resolved

should be fine by now

[icinga-migration]

Updated by mfriedrich on 2014-12-08 09:42:28 +00:00

• Project changed from 19 to Core, Classic UI, IDOUtils
• Category set to Classic UI
• Icinga Version set to 1
• OS Version set to any