(/home/sate/Testcases/c/cve/wireshark-1.2.0/fileset.c) |
| |
| 243 | | | fileset_add_dir(const char *fname) |
| 244 | | | { |
| 245 | | | WS_DIR *dir; |
| 246 | | | WS_DIRENT *file; |
| 247 | | | const char *name; |
| 248 | | | fileset_entry *entry; |
| 249 | | | GString *dirname; |
| 250 | | | gchar *fname_dup; |
| 251 | | | |
| 252 | | | |
| 253 | | | |
| 254 | | | fname_dup = get_dirname(g_strdup(fname)); |
| 255 | | | dirname = g_string_new(fname_dup); |
| 256 | | | g_free(fname_dup); |
| 257 | | | |
| 258 | | | set.dirname = g_strdup(dirname->str); |
| 259 | | | |
| 260 | | | dirname = g_string_append_c(dirname, G_DIR_SEPARATOR);
x /usr/include/glib-2.0/glib/gstring.h |
| |
156 | #define g_string_append_c(gstr,c) g_string_append_c_inline (gstr, c) |
| |
|
Useless Assignment
This code assigns the variable the same value it already had. |
|
| 261 | | | |
| 262 | | | |
| 263 | | | if(fileset_filename_match_pattern(fname)) { |
| 264 | | | |
| 265 | | | if ((dir = ws_dir_open(dirname->str, 0, NULL)) != NULL) { |
| 266 | | | while ((file = ws_dir_read_name(dir)) != NULL) { |
| 267 | | | name = ws_dir_get_name(file); |
| 268 | | | if(fileset_filename_match_pattern(name) && fileset_is_file_in_set(name, get_basename(fname))) { |
| 269 | | | fileset_add_file(dirname->str, name, strcmp(name, get_basename(fname))== 0 ); |
| 270 | | | } |
| 271 | | | } |
| 272 | | | |
| 273 | | | ws_dir_close(dir); |
| 274 | | | } |
| 275 | | | } else { |
| 276 | | | |
| 277 | | | entry = fileset_add_file(dirname->str, get_basename(fname), TRUE ); |
| 278 | | | |
| 279 | | | } |
| 280 | | | |
| 281 | | | g_string_free(dirname, TRUE ); |
| 282 | | | |
| 283 | | | |
| 284 | | | set.entries = g_list_sort(set.entries, fileset_sort_compare); |
| 285 | | | |
| 286 | | | fileset_update_dlg(); |
| 287 | | | } |
| |