diff options
author | Joseph Fernandes <josferna@redhat.com> | 2015-07-08 21:22:53 +0530 |
---|---|---|
committer | Dan Lambright <dlambrig@redhat.com> | 2015-07-09 12:11:25 -0700 |
commit | 3c25f33162c981d18bf78d1fc3536496670959e9 (patch) | |
tree | 2e3fa38c3728ad5b9ee251170838c6ef4345ed92 | |
parent | c134bdd83a781452eff8c3e4abce3da4a997daa8 (diff) |
libgfdb/ctr: Fixing rename or update of link
Fixing rename or updating of link in
1) libgfdb : handling return from updatelink
2) ctr xlator: correcting freeing of oldbase name in
hardlink list in the update link
Change-Id: I1789c492b81876123cd456051a9ae495e876ad46
BUG: 1241054
Signed-off-by: Joseph Fernandes <josferna@redhat.com>
Reviewed-on: http://review.gluster.org/11584
Tested-by: Joseph Fernandes
Reviewed-by: Dan Lambright <dlambrig@redhat.com>
Tested-by: Dan Lambright <dlambrig@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
-rw-r--r-- | libglusterfs/src/gfdb/gfdb_sqlite3_helper.c | 2 | ||||
-rw-r--r-- | xlators/features/changetimerecorder/src/ctr-xlator-ctx.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/libglusterfs/src/gfdb/gfdb_sqlite3_helper.c b/libglusterfs/src/gfdb/gfdb_sqlite3_helper.c index 40ee24c9627..68e0991bff2 100644 --- a/libglusterfs/src/gfdb/gfdb_sqlite3_helper.c +++ b/libglusterfs/src/gfdb/gfdb_sqlite3_helper.c @@ -829,7 +829,7 @@ gf_sql_insert_wind (gf_sql_connection_t *sql_conn, "Creating gfid string failed."); goto out; } - gf_sql_update_link (sql_conn, gfid_str, + ret = gf_sql_update_link (sql_conn, gfid_str, pargfid_str, gfdb_db_record->file_name, gfdb_db_record->file_path, diff --git a/xlators/features/changetimerecorder/src/ctr-xlator-ctx.c b/xlators/features/changetimerecorder/src/ctr-xlator-ctx.c index b8f6f0301d9..b5a54c0b5e4 100644 --- a/xlators/features/changetimerecorder/src/ctr-xlator-ctx.c +++ b/xlators/features/changetimerecorder/src/ctr-xlator-ctx.c @@ -213,7 +213,7 @@ ctr_update_hard_link (xlator_t *this, /* update the hard link */ gf_uuid_copy (ctr_hard_link->pgfid, pgfid); - GF_FREE (&ctr_hard_link->base_name); + GF_FREE (ctr_hard_link->base_name); ret = gf_asprintf(&ctr_hard_link->base_name, "%s", base_name); if (ret < 0) { gf_msg (this->name, GF_LOG_ERROR, 0, |