libaacs | branch: master | npzacs <[email protected]> | Wed Apr 25 14:51:29 2012 
+0300| [9ad9c476730ce41bdff249f0c45be7efb2eb1710] | committer: npzacs

Improved logging of fatal errors

> http://git.videolan.org/gitweb.cgi/libaacs.git/?a=commit;h=9ad9c476730ce41bdff249f0c45be7efb2eb1710
---

 src/libaacs/aacs.c |   24 ++++++++++++++----------
 src/libaacs/mmc.c  |    8 ++++----
 2 files changed, 18 insertions(+), 14 deletions(-)

diff --git a/src/libaacs/aacs.c b/src/libaacs/aacs.c
index 279aa9d..49b0946 100644
--- a/src/libaacs/aacs.c
+++ b/src/libaacs/aacs.c
@@ -146,10 +146,12 @@ static int _calc_mk(AACS *aacs, const char *path)
 
         mkb_close(mkb);
         X_FREE(buf);
-    }
 
-    DEBUG(DBG_AACS, "Error calculating media key!\n");
+        DEBUG(DBG_AACS | DBG_CRIT, "Error calculating media key. Missing right 
processing key ?\n");
+        return 0;
+    }
 
+    DEBUG(DBG_AACS | DBG_CRIT, "Error opening %s/AACS/MKB_RO.inf\n", path);
     return 0;
 }
 
@@ -334,7 +336,7 @@ static int _calc_uks(AACS *aacs, const char *path)
 
             } else {
                 aacs->num_uks = 0;
-                DEBUG(DBG_AACS, "Error reading number of unit keys!\n");
+                DEBUG(DBG_AACS | DBG_CRIT, "Error reading number of unit 
keys\n");
             }
 
             // Read keys
@@ -343,7 +345,7 @@ static int _calc_uks(AACS *aacs, const char *path)
 
                 file_seek(fp, f_pos, SEEK_SET);
                 if ((file_read(fp, buf, 16)) != 16) {
-                    DEBUG(DBG_AACS, "Unit key %d: read error\n", i);
+                    DEBUG(DBG_AACS | DBG_CRIT, "Unit key %d: read error\n", i);
                     aacs->num_uks = i;
                     break;
                 }
@@ -368,10 +370,12 @@ static int _calc_uks(AACS *aacs, const char *path)
         }
 
         file_close(fp);
-    }
 
-    DEBUG(DBG_AACS, "Could not calculate unit keys!\n");
+        DEBUG(DBG_AACS | DBG_CRIT, "Error reading unit keys\n");
+        return 0;
+    }
 
+    DEBUG(DBG_AACS | DBG_CRIT, "Error opening %s/AACS/Unit_Key_RO.inf\n", 
path);
     return 0;
 }
 
@@ -386,7 +390,7 @@ static int _calc_title_hash(const char *path, uint8_t 
*title_hash)
     f_name = str_printf("%s/AACS/Unit_Key_RO.inf", path);
 
     if (!(fp = file_open(f_name, "rb"))) {
-        DEBUG(DBG_AACS, "Failed to open unit key file: %s!\n", f_name);
+        DEBUG(DBG_AACS | DBG_CRIT, "Error opening unit key file %s\n", f_name);
         X_FREE(f_name);
         return 0;
     }
@@ -401,7 +405,7 @@ static int _calc_title_hash(const char *path, uint8_t 
*title_hash)
 
     if ((file_read(fp, ukf_buf, f_size)) != f_size) {
 
-        DEBUG(DBG_AACS, "Failed to read %"PRIu64" bytes from unit key 
file!\n", f_size);
+        DEBUG(DBG_AACS | DBG_CRIT, "Failed to read %"PRIu64" bytes from unit 
key file %s\n", f_size, f_name);
 
         file_close(fp);
         X_FREE(ukf_buf);
@@ -581,7 +585,7 @@ static int _load_config(AACS *aacs, const char 
*configfile_path)
     config_ok = keydbcfg_load_cert_file(aacs->cf) || config_ok;
 
     if (!config_ok) {
-        DEBUG(DBG_AACS, "No valid configuration files found!\n");
+        DEBUG(DBG_AACS | DBG_CRIT, "No valid AACS configuration files 
found\n");
     }
 
     return config_ok;
@@ -601,7 +605,7 @@ AACS *aacs_open(const char *path, const char 
*configfile_path)
     DEBUG(DBG_AACS, "Initializing libgcrypt...\n");
     if (!crypto_init())
     {
-        DEBUG(DBG_AACS, "Failed to initialize libgcrypt\n");
+        DEBUG(DBG_AACS | DBG_CRIT, "Failed to initialize libgcrypt\n");
         return NULL;
     }
 
diff --git a/src/libaacs/mmc.c b/src/libaacs/mmc.c
index b32f912..13184d6 100644
--- a/src/libaacs/mmc.c
+++ b/src/libaacs/mmc.c
@@ -390,7 +390,7 @@ MMC *mmc_open(const char *path)
 #ifdef HAVE_REALPATH
     char *file_path = malloc(PATH_MAX);
     if (!file_path || !realpath(path, file_path)) {
-        DEBUG(DBG_MMC, "Failed resolving path %s (%p)\n", path, mmc);
+        DEBUG(DBG_MMC | DBG_CRIT, "Failed resolving path %s (%p)\n", path, 
mmc);
         X_FREE(mmc);
         X_FREE(file_path);
         return NULL;
@@ -435,7 +435,7 @@ MMC *mmc_open(const char *path)
     }
 
     if (mmc->fd < 0) {
-        DEBUG(DBG_MMC, "Error opening LINUX MMC drive mounted to %s (%p)\n", 
file_path, mmc);
+        DEBUG(DBG_MMC | DBG_CRIT, "Error opening LINUX MMC drive mounted to %s 
(%p)\n", file_path, mmc);
         X_FREE(mmc);
     }
 
@@ -450,7 +450,7 @@ MMC *mmc_open(const char *path)
     UINT type = GetDriveType(drive);
 
     if (type != DRIVE_CDROM) {
-        DEBUG(DBG_MMC, "Drive %s is not CD/DVD drive !\n", drive);
+        DEBUG(DBG_MMC | DBG_CRIT, "Drive %s is not CD/DVD drive\n", drive);
         X_FREE(mmc);
         return NULL;
     }
@@ -473,7 +473,7 @@ MMC *mmc_open(const char *path)
 
 #else
 
-    DEBUG(DBG_MMC, "No MMC drive support !\n");
+    DEBUG(DBG_MMC | DBG_CRIT, "No MMC drive support\n");
     X_FREE(mmc);
 
 #endif

_______________________________________________
libaacs-devel mailing list
[email protected]
http://mailman.videolan.org/listinfo/libaacs-devel

Reply via email to