From 5232542d7ed273d800445ac8ebf7774c28513865 Mon Sep 17 00:00:00 2001
From: "Safey A.Halim" <safey.allah@gmail.com>
Date: Fri, 25 Feb 2011 10:13:28 +0000
Subject: [PATCH] codesonar problems fixed

---
 src/monkey/gdbmi_parse.c | 42 ++++++++++++++++++++++------------------
 1 file changed, 23 insertions(+), 19 deletions(-)

diff --git a/src/monkey/gdbmi_parse.c b/src/monkey/gdbmi_parse.c
index e3767c418..678276ba3 100644
--- a/src/monkey/gdbmi_parse.c
+++ b/src/monkey/gdbmi_parse.c
@@ -50,15 +50,12 @@ int mi_get_cstring_r(mi_results *r, const char *str, const char **end)
  /* Meassure. */
  for (s=str, len=0; *s && !EndOfStr(s); s++)
     {
+	 if (!*s) {
+		 mi_error = MI_PARSER;
+		 return 0;
+	 }
      if (*s=='\\')
-       {
-        if (!*s)
-          {
-           mi_error=MI_PARSER;
-           return 0;
-          }
-        s++;
-       }
+    	 s++;
      len++;
     }
  /* Copy. */
@@ -117,8 +114,10 @@ char *mi_get_var_name(const char *str, const char **end)
  l=s-str;
  r=mi_malloc(l+1);
  /* Copy. */
- memcpy(r,str,l);
- r[l]=0;
+ if (NULL != r) {
+	 memcpy(r,str,l);
+	 r[l]=0;
+ }
  if (end)
     *end=s+1;
  return r;
@@ -1086,16 +1085,16 @@ int mi_res_changelist(mi_h *h, mi_gvar_chg **changed)
              n->name=r->v.cstr;
              r->v.cstr=NULL;
             }
-          else if (strcmp(r->var,"in_scope")==0)
+          else if ((NULL != n) && (strcmp(r->var,"in_scope")==0))
             {
              n->in_scope=strcmp(r->v.cstr,"true")==0;
             }
-          else if (strcmp(r->var,"new_type")==0)
+          else if ((NULL != n) && (strcmp(r->var,"new_type")==0))
             {
              n->new_type=r->v.cstr;
              r->v.cstr=NULL;
             }
-          else if (strcmp(r->var,"new_num_children")==0)
+          else if ((NULL != n) && (strcmp(r->var,"new_num_children")==0))
             {
              n->new_num_children=atoi(r->v.cstr);
             }
@@ -1133,7 +1132,7 @@ int mi_get_children(mi_results *ch, mi_gvar *v)
           return 0;
        if (!v->child)
           v->child=aux;
-       else
+       else if (NULL != cur)
           cur->next=aux;
        cur=aux;
        cur->parent=v;
@@ -1738,9 +1737,12 @@ mi_chg_reg *mi_parse_list_regs(mi_results *r, int *how_many)
           cur=cur->next=mi_alloc_chg_reg();
        else
           first=cur=mi_alloc_chg_reg();
-       cur->name=c->v.cstr;
-       cur->reg=cregs++;
-       c->v.cstr=NULL;
+       
+       if (NULL != cur) {
+	       cur->name=c->v.cstr;
+	       cur->reg=cregs++;
+	       c->v.cstr=NULL;
+       }
       }
     c=c->next;
    }
@@ -1891,12 +1893,14 @@ mi_chg_reg *mi_parse_reg_values_l(mi_results *r, int *how_many)
             {
              if (strcmp(c->var,"number")==0)
                {
-                cur->reg=atoi(c->v.cstr);
+            	if (NULL != cur) 
+            		cur->reg=atoi(c->v.cstr);
                 (*how_many)++;
                }
              else if (strcmp(c->var,"value")==0)
                {
-                cur->val=c->v.cstr;
+            	if (NULL != cur)
+            		 cur->val=c->v.cstr;
                 c->v.cstr=NULL;
                }
             }
-- 
2.25.1