Merge lp:~akopytov/percona-xtrabackup/bug900175 into lp:percona-xtrabackup/2.0

Proposed by Alexey Kopytov
Status: Merged
Approved by: Stewart Smith
Approved revision: no longer in the source branch.
Merged at revision: 395
Proposed branch: lp:~akopytov/percona-xtrabackup/bug900175
Merge into: lp:percona-xtrabackup/2.0
Diff against target: 24 lines (+4/-1)
1 file modified
src/xtrabackup.c (+4/-1)
To merge this branch: bzr merge lp:~akopytov/percona-xtrabackup/bug900175
Reviewer Review Type Date Requested Status
Stewart Smith (community) Approve
Review via email: mp+92965@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alexey Kopytov (akopytov) wrote :
Revision history for this message
Alexey Kopytov (akopytov) wrote :

Jenkins revealed a problem with the fix. URL for the new version: http://jenkins.percona.com/view/Percona%20Xtrabackup/job/percona-xtrabackup-param/147/

Revision history for this message
Stewart Smith (stewart) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/xtrabackup.c'
2--- src/xtrabackup.c 2012-02-14 10:38:07 +0000
3+++ src/xtrabackup.c 2012-02-14 12:52:20 +0000
4@@ -870,6 +870,8 @@
5 fil_node_t *
6 datafiles_iter_next(datafiles_iter_t *it)
7 {
8+ fil_node_t *new_node;
9+
10 os_mutex_enter(it->mutex);
11
12 if (it->node == NULL) {
13@@ -896,9 +898,10 @@
14 it->node = UT_LIST_GET_FIRST(it->space->chain);
15
16 end:
17+ new_node = it->node;
18 os_mutex_exit(it->mutex);
19
20- return it->node;
21+ return new_node;
22 }
23
24 static

Subscribers

People subscribed via source and target branches