[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#7960) [LMDB] wrong pointer used in mdb_cassert()



jcd@tribudubois.net wrote:
> Full_Name: Jean-Christophe Dubois
> Version: 2.4.40
> OS: Linux
> URL: ftp://ftp.openldap.org/incoming/
> Submission from: (NULL) (78.235.240.156)
>
>
> In mdb_node_move() csrc is passed to mdb_cassert() at line 7396 when it seems it
> should be cdst (as the operation is on cdst).
>
> https://gitorious.org/mdb/mdb/source/56c2c160be19c555e4c42e459c8608ffaae7b150:libraries/liblmdb/mdb.c#L7396

Irrelevant. The cursor is only passed to provide an env pointer, and both 
cursors point to the same env. Closing this ITS.
>
> Patch available at URL below:
>
> https://github.com/jcdubois/lmdb/commit/41ed03c4584ac46dc233dcf60f93addb09962093
>
> JC
>
>


-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/