fix delete_all_host_comment, unschedule_downtime and delete_downtime_by_hostname_service_description_start_time_comment.

[Dorian Guillois]

On Thu, Oct 6, 2011 at 2:37 PM, Andreas Ericsson <ae at op5.se> wrote:
> On 10/06/2011 01:07 PM, Dorian Guillois wrote:
>> Hi everybody,
>>
>> Attached patches fix some bugs on functions delete_all_host_comment,
>> unschedule_downtime and
>> delete_downtime_by_hostname_service_description_start_time_comment. In
>> this 3 functions a pointer on a deleted object is used.
>>
>
> Good catch.
>
> There's a bug in your patches though. In all cases you point next_item to
> this_item->next, but then iterate it using next_item->next, meaning you'll
> skip every other item.
>
> If you resend corrected patches I'll apply them. Thanks
>
> --
> Andreas Ericsson                   andreas.ericsson at op5.se
> OP5 AB                             www.op5.se
> Tel: +46 8-230225                  Fax: +46 8-230231
>
> Considering the successes of the wars on alcohol, poverty, drugs and
> terror, I think we should give some serious thought to declaring war
> on peace.
>

I'm so sorry. I was a little too fast.
delete_all_host_comment.patch was good, I resend the other file.

Best regards,

-- 
Dorian GUILLOIS | Développeur

dguillois at merethis.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: delete_and_unschedule_downtime.patch
Type: text/x-diff
Size: 1774 bytes
Desc: not available
URL: <https://www.monitoring-lists.org/archive/developers/attachments/20111006/0978b5e7/attachment.patch>
-------------- next part --------------
------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1
-------------- next part --------------
_______________________________________________
Nagios-devel mailing list
Nagios-devel at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nagios-devel