On Thu 28 Mar 2019 11:33:55 AM CET, Vladimir Sementsov-Ogievskiy wrote:
>>> All three functions that handle the BdrvChild.frozen attribute walk
>>> the backing chain from 'bs' to 'base' and stop either when 'base' is
>>> found or at the end of the chain if 'base' is NULL.
>>>
>>> However if 'base'
28.03.2019 13:27, Vladimir Sementsov-Ogievskiy wrote:
> 26.03.2019 20:07, Alberto Garcia wrote:
>> All three functions that handle the BdrvChild.frozen attribute walk
>> the backing chain from 'bs' to 'base' and stop either when 'base' is
>> found or at the end of the chain if 'base' is NULL.
>>
26.03.2019 20:07, Alberto Garcia wrote:
> All three functions that handle the BdrvChild.frozen attribute walk
> the backing chain from 'bs' to 'base' and stop either when 'base' is
> found or at the end of the chain if 'base' is NULL.
>
> However if 'base' is not found then the functions return
28.03.2019 13:04, Alberto Garcia wrote:
> On Thu 28 Mar 2019 10:45:51 AM CET, Vladimir Sementsov-Ogievskiy wrote:
>>> -for (i = bs; i != base && i->backing; i = backing_bs(i)) {
>>> -if (i->backing->frozen) {
>>> +for (i = bs; i != base; i = backing_bs(i)) {
>>> +if
On Thu 28 Mar 2019 10:45:51 AM CET, Vladimir Sementsov-Ogievskiy wrote:
>> -for (i = bs; i != base && i->backing; i = backing_bs(i)) {
>> -if (i->backing->frozen) {
>> +for (i = bs; i != base; i = backing_bs(i)) {
>> +if (i->backing && i->backing->frozen) {
>
> may be a bit
26.03.2019 20:07, Alberto Garcia wrote:
> All three functions that handle the BdrvChild.frozen attribute walk
> the backing chain from 'bs' to 'base' and stop either when 'base' is
> found or at the end of the chain if 'base' is NULL.
>
> However if 'base' is not found then the functions return
All three functions that handle the BdrvChild.frozen attribute walk
the backing chain from 'bs' to 'base' and stop either when 'base' is
found or at the end of the chain if 'base' is NULL.
However if 'base' is not found then the functions return without
errors as if it was NULL.
This is wrong: