[ros-diffs] [reactos] 07/07: [SDBINST] Restore file attributes before deleting

[Mark Jansen]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=20c98b31443647316e06c03c4ff336251c5b844a

commit 20c98b31443647316e06c03c4ff336251c5b844a
Author:     Mark Jansen <mark.jan...@reactos.org>
AuthorDate: Sat May 8 15:49:28 2021 +0200
Commit:     Mark Jansen <mark.jan...@reactos.org>
CommitDate: Sat May 8 15:49:28 2021 +0200

    [SDBINST] Restore file attributes before deleting
---
 base/applications/sdbinst/sdbinst.c | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/base/applications/sdbinst/sdbinst.c 
b/base/applications/sdbinst/sdbinst.c
index fb807e80136..68f31a477bb 100644
--- a/base/applications/sdbinst/sdbinst.c
+++ b/base/applications/sdbinst/sdbinst.c
@@ -579,7 +579,7 @@ SdbUninstall(
     if (pdb == NULL)
     {
         wprintf(L"Can't open database %ls\n", sdbPath);
-        goto end;
+        return FALSE;
     }
 
     tagDb = SdbFindFirstTag(pdb, TAGID_ROOT, TAG_DATABASE);
@@ -590,7 +590,7 @@ SdbUninstall(
     }
 
     //
-    if (!GetSdbGuid(pdb, tagDb, &dbGuid))    
+    if (!GetSdbGuid(pdb, tagDb, &dbGuid))
     {
         wprintf(L"GetSdbGuid error\n");
         goto end;
@@ -620,29 +620,32 @@ SdbUninstall(
     }
 
     SdbCloseDatabase(pdb);
+    pdb = NULL;
 
     hres = DeleteUninstallKey(sdbName);
     if (FAILED(hres))
     {
         wprintf(L"Remove uninstall key fail\n");
-        //goto end;
     }
 
     if (!SdbUnregisterDatabase(&dbGuid))
     {
         wprintf(L"SdbUnregisterDatabase\n");
-        goto end;
+        return FALSE;
     }
 
-    if (!DeleteFile(sdbPath))
+    SetFileAttributesW(sdbPath, FILE_ATTRIBUTE_NORMAL);
+    if (!DeleteFileW(sdbPath))
     {
         wprintf(L"Remove file fail 0x%08X\n", GetLastError());
-        goto end;
+        return FALSE;
     }
 
     res = TRUE;
 
 end:
+    if (pdb)
+        SdbCloseDatabase(pdb);
     return res;
 }
 
@@ -871,7 +874,7 @@ int _tmain(int argc, LPWSTR argv[])
                 }
 
                 nameSdbStr = argv[i];
-                wprintf(L"guidSdbStr %ls\n", nameSdbStr);
+                wprintf(L"nameSdbStr %ls\n", nameSdbStr);
 
                 isUninstByName = TRUE;
                 isInstall = FALSE;