Text  |   XML   |   Visible Warnings:

Irssi : Irssi analysis 1 : Null Test After Dereference  at Expando.c:125

Categories: LANG.STRUCT.NTAD
Warning ID: 2305.2314
Procedure: expando_signals_add_hash
Trace: View
Modified: Wed Sep 2 12:01:40 2009   show details
 
Priority: None
State: None
Finding: None
Owner: None
  edit properties

Legend [ X ]
Warning Location
Contributes
Parse Error
Other Warning
Two or More Loop Iterations
On Execution Path
Comment
Macro
Preprocessor
Include
Keyword
Preprocessed Away
Legend
Warning Location Contributes Parse Error
Other Warning Two or More Loop Iterations On Execution Path
Comment Macro Preprocessor
Include Keyword Preprocessed Away

Source  |  Language: C Hide Legend     
LineSource
  /u1/paul/SATE/2010/c/irssi/irssi-0.8.14/src/perl/common/Expando.c
  Enter expando_signals_add_hash
112 static void expando_signals_add_hash(const char *key, SV *signals) 
113 {
114         HV *hv;
115         HE *he;
116         I32 len;
117         const char *argstr;
118         ExpandoArg arg;
119  
120         if (!is_hvref(signals)) {
121                 croak("Usage: Irssi::expando_create(key, func, hash)");
122                 return;
123         }
124  
125         hv = hvref(signals);     /* Null Test After Dereference */
126         hv_iterinit(hv);
127         while ((he = hv_iternext(hv)) != NULL) {
128                 SV *argsv = HeVAL(he);
129                 argstr = SvPV(argsv, PL_na);
130  
131                 if (g_ascii_strcasecmp(argstr, "none") == 0)
132                         arg = EXPANDO_ARG_NONE;
133                 else if (g_ascii_strcasecmp(argstr, "server") == 0)
134                         arg = EXPANDO_ARG_SERVER;
135                 else if (g_ascii_strcasecmp(argstr, "window") == 0)
136                         arg = EXPANDO_ARG_WINDOW;
137                 else if (g_ascii_strcasecmp(argstr, "windowitem") == 0)
138                         arg = EXPANDO_ARG_WINDOW_ITEM;
139                 else if (g_ascii_strcasecmp(argstr, "never") == 0)
140                         arg = EXPANDO_NEVER;
141                 else {
142                         croak("Unknown signal type: %s", argstr);
143                         break;
144                 }
145                 expando_add_signal(key, hv_iterkey(he, &len), arg);
146         }
147 } 



Change Warning 2305.2314 : Null Test After Dereference

Priority:
State:
Finding:
Owner:
Note: