Changeset 1104 in MondoRescue for branches/stable/mondo/src/lib

Timestamp:

Feb 7, 2007, 8:00:23 PM (17 years ago)

Author:

Bruno Cornec

Message:

Improvement of the low level library (Usage of LINE and FILE and simplified interfaces using macros)

Location:

branches/stable/mondo/src/lib

Files:

• mr_err.c (modified) (2 diffs)
• mr_mem.c (modified) (6 diffs)
• mr_msg.c (modified) (2 diffs)

branches/stable/mondo/src/lib/mr_err.c

@@ -27 +27 @@
         /* We have to properly end newt */
         /* We have to remind people of log files */
+        mr_msg_close();
 }
 
@@ -38 +39 @@
     exit(errorcode);
 }
-
-void mr_log_exit(int errorcode, const char *message) {
-    mr_msg(0, message);
-    mr_exit(errorcode, message);
-}

branches/stable/mondo/src/lib/mr_mem.c

@@ -22 +22 @@
 /*
  * Function that frees memory if necessary
+ * A pointer to the memory pointed is passed to it.
+ * *allocated variable points then to the original content 
+ * pointed to by the caller
  */
-void mr_free(void *allocated) {
+void mr_free_int(void **allocated, int line, const char *file) {
 
     /* free man pages says that if allocated is NULL 
      * nothing happens 
      */
-    free(allocated);
-    allocated = NULL;
+    if (*allocated != NULL) {
+        free(*allocated);
+        *allocated = NULL;
+    } else {
+        mr_msg_int(0,line,file,"Attempt to reference NULL pointer\nExiting...");
+        mr_exit(-1,"Attempt to reference NULL pointer");
+    }
 }
 
 /* encapsulation function for malloc */
-void *mr_malloc(size_t size) {
+void *mr_malloc_int(size_t size, int line, const char *file) {
     
     void *ret;
@@ -39 +47 @@
     ret = malloc(size);
     if (ret == NULL) {
-        mr_log_exit(-1,"Unable to alloc memory in mr_malloc\nExiting...");
+        mr_msg_int(0,line,file,"Unable to alloc memory in mr_malloc\nExiting...");
+        mr_exit(-1,"Unable to alloc memory in mr_malloc");
     }
     return(ret);
@@ -45 +54 @@
 
 /* encapsulation function for getline */
-void mr_getline(char **lineptr, size_t *n, FILE *stream) {
+void mr_getline_int(char **lineptr, size_t *n, FILE *stream, int line, const char *file) {
     
     ssize_t ret;
@@ -51 +60 @@
     ret = getline(lineptr,n,stream);
     if (ret == -1) {
-        mr_log_exit(-1,"Unable to alloc memory in mr_getline\nExiting...");
+        mr_msg_int(0,line,file,"Unable to alloc memory in mr_getline\nExiting...",line,file);
+        mr_exit(-1,"Unable to alloc memory in mr_getline");
     }
 }
 
 /* encapsulation function for asprintf */
-void mr_asprintf(char **strp, const char *fmt, ...) {
+void mr_asprintf_int(char **strp, int line, const char *file, const char *fmt, ...) {
 
     int res = 0;
@@ -64 +74 @@
     res = vasprintf(strp, fmt, args);
     if (res == -1) {
-        mr_log_exit(-1,"Unable to alloc memory in mr_asprintf\nExiting...");
+        mr_msg_int(0,line,file,"Unable to alloc memory in mr_asprintf\nExiting...",line,file);
+        mr_exit(-1,"Unable to alloc memory in mr_asprintf");
     }
     va_end(args);
@@ -73 +84 @@
  * freeing it before in any case
  */
-void mr_allocstr(char *alloc, const char *orig) {
+void mr_allocstr_int(char *alloc, const char *orig, int line, const char *file) {
 
-    mr_free((void *)alloc); 
-    mr_asprintf(&alloc, orig);
+    mr_free_int((void **)&alloc, line, file); 
+    mr_asprintf_int(&alloc, line, file, orig);
 }

branches/stable/mondo/src/lib/mr_msg.c

@@ -52 +52 @@
  * Function that log a message. Not called directly 
  * but through other functions
+ * fmt needs to be just before ...
  */
-void mr_msg(int debug, const char *fmt, ...) {
+void mr_msg_int(int debug, int line, const char *file, const char *fmt, ...) {
 
     int i = 0;
@@ -74 +75 @@
             for (i = 1; i < debug; i++)
                 fprintf(fout, "  ");
-            fprintf(fout, "%s->%s#%d: ", __FILE__, __FUNCTION__, __LINE__);
+            fprintf(fout, "%s #%d: ", file, line);
         }
         va_start(args,fmt);