cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daan Hoogland <daan.hoogl...@gmail.com>
Subject Re: null deref
Date Fri, 05 Sep 2014 14:19:41 GMT
As we don't have a good branch and merge strategy yet you probably should
have cherry-picked and the message would have been copied.


On Fri, Sep 5, 2014 at 4:18 PM, Daan Hoogland <daan.hoogland@gmail.com>
wrote:

> so now we are not giving coverity credit for finding it. let them sue us.
> But seriously thanks for your admission of guilt. I think a good discipline
> at this is worth something on the long run.
>
>
> On Fri, Sep 5, 2014 at 4:08 PM, Will Stevens <wstevens@cloudops.com>
> wrote:
>
>> I have merged the change into master.  I forgot to reference the
>> 'CID-123719' in the commit note though.
>>
>>
>> *Will STEVENS*
>> Lead Developer
>>
>> *CloudOps* *| *Cloud Solutions Experts
>> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
>> w cloudops.com *|* tw @CloudOps_
>>
>>
>> On Fri, Sep 5, 2014 at 10:02 AM, Will Stevens <wstevens@cloudops.com>
>> wrote:
>>
>>> Ok, thanks...
>>>
>>> ws
>>>
>>>
>>> *Will STEVENS*
>>> Lead Developer
>>>
>>> *CloudOps* *| *Cloud Solutions Experts
>>> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
>>> w cloudops.com *|* tw @CloudOps_
>>>
>>>
>>> On Fri, Sep 5, 2014 at 10:02 AM, Daan Hoogland <daan.hoogland@gmail.com>
>>> wrote:
>>>
>>>> go ahead and cherry-pick I will merge in 4.4.
>>>>
>>>>
>>>> On Fri, Sep 5, 2014 at 3:52 PM, Will Stevens <wstevens@cloudops.com>
>>>> wrote:
>>>>
>>>>> Yes, I am fine with that change.  Should I add it to master?
>>>>>
>>>>> Cheers,
>>>>>
>>>>> Will
>>>>>
>>>>>
>>>>> *Will STEVENS*
>>>>> Lead Developer
>>>>>
>>>>> *CloudOps* *| *Cloud Solutions Experts
>>>>> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
>>>>> w cloudops.com *|* tw @CloudOps_
>>>>>
>>>>>
>>>>> On Fri, Sep 5, 2014 at 8:54 AM, Daan Hoogland <daan.hoogland@gmail.com
>>>>> > wrote:
>>>>>
>>>>>> H Will, coverity complained about a few lines of yours. please have
a
>>>>>> look at hotfix/4.4-CID1237195 and tell me you agree or not. I think
it
>>>>>> should go in master as well.
>>>>>>
>>>>>> --
>>>>>> Daan
>>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> Daan
>>>>
>>>
>>>
>>
>
>
> --
> Daan
>



-- 
Daan

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message