From: Dmitry Lenev
Date: December 22 2010 1:19pm
Subject: Re: bzr commit into mysql-trunk-bugfixing branch
	(alexander.nozdrin:3465) Bug#59022
List-Archive: http://lists.mysql.com/commits/127505
Message-Id: <20101222131957.GB13835@mockturtle>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii

Hello Alik!

* Alexander Nozdrin <alexander.nozdrin@stripped> [10/12/22 15:52]:
> #At file:///home/alik/MySQL/bzr/00.build/mysql-trunk-bugfixing/ based on revid:tor.didriksen@stripped
> 
>  3465 Alexander Nozdrin	2010-12-22
>       A patch for Bug#59022 (Valgrind warning in write_delayed() after InnoDB merge).
>       
>       Free allocated bitmap.
> 
>     modified:
>       sql/sql_insert.cc

I suggest to provide a more elaborate ChangeSet comment.

Something like:

"Fix memory leak introduced by WL#5092 'RBR: Options for writing
 partial or full row images in RBR events'.
 Free memory which is allocated for column bitmap associated with
 each row to be processed by delayed insert handler thread."

IMO will do the trick.

> === modified file 'sql/sql_insert.cc'
> --- a/sql/sql_insert.cc	2010-11-29 16:27:58 +0000
> +++ b/sql/sql_insert.cc	2010-12-22 12:46:55 +0000
> @@ -1843,11 +1843,14 @@ public:
>      : record(0), dup(dup_arg), ignore(ignore_arg), log_query(log_query_arg),
>        binlog_rows_query_log_events(FALSE),
>        forced_insert_id(0), query(query_arg), time_zone(0)
> -    {}
> +    {
> +      write_set.bitmap= NULL;
> +    }
>    ~delayed_row()
>    {
>      my_free(query.str);
>      my_free(record);
> +    my_free(write_set.bitmap);
>    }
>  };

Otherwise I am OK with the above patch and think that it can be pushed.

-- 
Dmitry Lenev, Software Developer
Oracle Development SPB/MySQL, www.mysql.com

Are you MySQL certified?  http://www.mysql.com/certification