Re: Fix crash in mnesia_controller due to a function_clause exception from is_tab_blocked/1

Ahmed Omar <[email protected]>
Newsgroups gmane.comp.lang.erlang.patches
Message-ID <CALxv_xbrio3nXy3Eu4nn+5xJgEUat=BV9eDxfFFatn+38=TiqA@mail.gmail.com>
Thanks Fredrik for your comments, to avoid confusing commits,
Please fetch from

git fetch git://github.com/spawnthink/otp.git
fix_mnesia_block_table_exception

https://github.com/spawnthink/otp/compare/fix_mnesia_block_table_exception
https://github.com/spawnthink/otp/compare/fix_mnesia_block_table_exception.patch


On Thu, May 2, 2013 at 10:05 AM, Ahmed Omar <[email protected]> wrote:

> Hello Fredrik,
> Could you please refetch, I removed an extra call to val(Var)
> https://github.com/spawnthink/otp/compare/fix_mnesia_block_table
>
>
> On Thu, May 2, 2013 at 9:12 AM, Fredrik <[email protected]> wrote:
>
>>  On 05/01/2013 03:49 PM, Ahmed Omar wrote:
>>
>> git fetch git://github.com/spawnthink/otp.git fix_mnesia_block_table
>>
>>  Fix crash in mnesia_controller due to a function_clause exception from
>> is_tab_blocked/1
>>
>>  If mnesia_controller:block_table/1 is called twice for same table,
>> the new stored value of {Tab, where_to_commit} will be {blocked,
>> {blocked,List}}.
>> This will result in an exception when calling is_tab_blocked/1.
>> mnesia_conttoller:block_table/1 should check if the table is already
>> blocked.
>>
>>  https://github.com/spawnthink/otp/compare/fix_mnesia_block_table.patch
>>  https://github.com/spawnthink/otp/compare/fix_mnesia_block_table
>>
>>  Best Regards,
>> Ahmed Omar
>>
>>
>> _______________________________________________
>> erlang-patches mailing [email protected]://erlang.org/mailman/listinfo/erlang-patches
>>
>>  Hello Ahmed,
>> I've fetched your patch and it is currently in the 'pu' branch.
>>
>> --
>>
>> BR Fredrik Gustafsson
>> Erlang OTP Team
>>
>>
>

_______________________________________________
erlang-patches mailing list
[email protected]
http://erlang.org/mailman/listinfo/erlang-patches
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.