- Remove duplicated code
- Insert blank line for clarity
Change-Id: Ie9778d1329e418d03741948c893ce2a070f2c4fe
Signed-off-by: Ilho Kim <ilho159.kim@samsung.com>
+static int __backup_db(const char *src_path, const char *dest_path) {
+ if (__backup_file(src_path, dest_path) != 0)
+ return -1;
+
+ if (__set_db_permission(src_path) != 0)
+ return -1;
+
+ return 0;
+}
+
static int __make_backup_dbs() {
int ret = 0;
char parser_db_bck[BUF_SIZE];
static int __make_backup_dbs() {
int ret = 0;
char parser_db_bck[BUF_SIZE];
snprintf(cert_db_bck, BUF_SIZE, "%s.bck", CERT_DBPATH);
snprintf(cert_db_journal_bck, BUF_SIZE, "%s.bck", JOURNAL_CERT_DBPATH);
snprintf(cert_db_bck, BUF_SIZE, "%s.bck", CERT_DBPATH);
snprintf(cert_db_journal_bck, BUF_SIZE, "%s.bck", JOURNAL_CERT_DBPATH);
- tryvm_if(__backup_file(DBPATH, parser_db_bck) == -1,
+ tryvm_if(__backup_db(DBPATH, parser_db_bck) == -1,
ret = -1, "Fail to backup [%s] to [%s]\n",
DBPATH, parser_db_bck);
ret = -1, "Fail to backup [%s] to [%s]\n",
DBPATH, parser_db_bck);
- tryvm_if(__set_db_permission(DBPATH) == -1,
- ret = -1, "Fail to set db permission\n");
- tryvm_if(__backup_file(JOURNAL_DBPATH, parser_db_journal_bck) == -1,
+ tryvm_if(__backup_db(JOURNAL_DBPATH, parser_db_journal_bck) == -1,
ret = -1, "Fail to backup [%s] to [%s]\n",
JOURNAL_DBPATH, parser_db_journal_bck);
ret = -1, "Fail to backup [%s] to [%s]\n",
JOURNAL_DBPATH, parser_db_journal_bck);
- tryvm_if(__set_db_permission(JOURNAL_DBPATH) == -1,
- ret = -1, "Fail to set db permission\n");
- tryvm_if(__backup_file(CERT_DBPATH, cert_db_bck) == -1,
+ tryvm_if(__backup_db(CERT_DBPATH, cert_db_bck) == -1,
ret = -1, "Fail to backup [%s] to [%s]\n",
CERT_DBPATH, cert_db_bck);
ret = -1, "Fail to backup [%s] to [%s]\n",
CERT_DBPATH, cert_db_bck);
- tryvm_if(__set_db_permission(CERT_DBPATH) == -1,
- ret = -1, "Fail to set db permission\n");
- tryvm_if(__backup_file(JOURNAL_CERT_DBPATH, cert_db_journal_bck) == -1,
+ tryvm_if(__backup_db(JOURNAL_CERT_DBPATH, cert_db_journal_bck) == -1,
ret = -1, "Fail to backup [%s] to [%s]\n",
JOURNAL_CERT_DBPATH, cert_db_journal_bck);
ret = -1, "Fail to backup [%s] to [%s]\n",
JOURNAL_CERT_DBPATH, cert_db_journal_bck);
- tryvm_if(__set_db_permission(JOURNAL_CERT_DBPATH) == -1,
- ret = -1, "Fail to set db permission\n");
static void __remove_backup_path(const char *origin_path) {
char backup_path[BUF_SIZE];
static void __remove_backup_path(const char *origin_path) {
char backup_path[BUF_SIZE];
snprintf(backup_path, BUF_SIZE, "%s.bck", origin_path);
if (remove(backup_path))
snprintf(backup_path, BUF_SIZE, "%s.bck", origin_path);
if (remove(backup_path))
__make_preload_rw_list(preload_rw_table);
g_hash_table_destroy(preload_rw_table);
__remove_backup_dbs();
__make_preload_rw_list(preload_rw_table);
g_hash_table_destroy(preload_rw_table);
__remove_backup_dbs();