fix delete_all_host_comment, unschedule_downtime and delete_downtime_by_hostname_service_description_start_time_comment.

[Andreas Ericsson]

On 10/06/2011 02:53 PM, Dorian Guillois wrote:
> 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.
> 

Applied. 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.

------------------------------------------------------------------------------
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