Attachment 647713 Details for Bug 878001 – List of all defects added in irqbalance-1.0.4-3.el6

List of all defects added in irqbalance-1.0.4-3.el6

irqbalance.added.err (text/plain), 6.98 KB, created by Ondrej Oprala on 2012-11-19 12:21:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ondrej Oprala

Created: 2012-11-19 12:21:53 UTC

Size: 6.98 KB

patch

obsolete

>Error: NULL_RETURNS (CWE-476):
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:418: returned_null: Function "g_list_find_custom(GList *, gconstpointer, GCompareFunc)" returns null (checked 6 out of 7 times).
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:371:3: cond_true: Condition "func != NULL", taking true branch
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:371:3: if_fallthrough: Falling through to end of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:371:3: if_end: End of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:373:3: cond_true: Condition "list", taking true branch
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:375:7: cond_false: Condition "!(*func)(list->data, data)", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:376:2: if_end: End of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:378:5: loop: Jumping back to the beginning of the loop
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:373:3: loop_begin: Jumped back to beginning of loop
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:373:3: cond_false: Condition "list", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:378:5: loop_end: Reached end of loop
>/builddir/build/BUILD/irqbalance-1.0.4/glib-local/glist.c:380:3: return_null: Explicitly returning null.
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:74: example_assign: Assigning: "entry" = return value from "g_list_find_custom(banned_irqs, &find, compare_ints(gconstpointer, gconstpointer))".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:75: example_checked: "entry" has its value checked in "entry".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:112: example_assign: Assigning: "entry" = return value from "g_list_find_custom(interrupts_db, &find, compare_ints(gconstpointer, gconstpointer))".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:113: example_checked: "entry" has its value checked in "entry".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:119: example_assign: Assigning: "entry" = return value from "g_list_find_custom(banned_irqs, &find, compare_ints(gconstpointer, gconstpointer))".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:120: example_checked: "entry" has its value checked in "entry".
>/builddir/build/BUILD/irqbalance-1.0.4/cputree.c:425: example_assign: Assigning: "entry" = return value from "g_list_find_custom(cpus, &find, compare_cpus(gconstpointer, gconstpointer))".
>/builddir/build/BUILD/irqbalance-1.0.4/cputree.c:427: example_checked: "entry" has its value checked in "entry".
>/builddir/build/BUILD/irqbalance-1.0.4/numa.c:182: example_assign: Assigning: "entry" = return value from "g_list_find_custom(numa_nodes, &find, compare_node(gconstpointer, gconstpointer))".
>/builddir/build/BUILD/irqbalance-1.0.4/numa.c:183: example_checked: "entry" has its value checked in "entry".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:418: var_assigned: Assigning: "entry" = null return value from "g_list_find_custom(GList *, gconstpointer, GCompareFunc)".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:419: dereference: Dereferencing a null pointer "entry".
>
>Error: NULL_RETURNS (CWE-476):
>/builddir/build/BUILD/irqbalance-1.0.4/numa.c:63: cond_false: Condition "!new", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/numa.c:64: if_end: End of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/numa.c:66: returned_null: Function "fopen(char const * restrict, char const * restrict)" returns null.
>/builddir/build/BUILD/irqbalance-1.0.4/numa.c:66: var_assigned: Assigning: "f" = null return value from "fopen(char const * restrict, char const * restrict)".
>/builddir/build/BUILD/irqbalance-1.0.4/numa.c:67: dereference: Dereferencing a pointer that might be null "f" when calling "ferror(FILE *)".
>
>Error: PASS_BY_VALUE:
>/builddir/build/BUILD/irqbalance-1.0.4/cputree.c:61: pass_by_value: Passing parameter package_mask of type cpumask_t (size 512 bytes) by value.
>
>Error: PASS_BY_VALUE:
>/builddir/build/BUILD/irqbalance-1.0.4/cputree.c:103: pass_by_value: Passing parameter cache_mask of type cpumask_t (size 512 bytes) by value.
>
>Error: PASS_BY_VALUE:
>/builddir/build/BUILD/irqbalance-1.0.4/activate.c:35: pass_by_value: Passing parameter applied_mask of type cpumask_t (size 512 bytes) by value.
>
>Error: RESOURCE_LEAK (CWE-404):
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:375: alloc_fn: Storage is returned from allocation function "calloc(size_t, size_t)".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:375: var_assign: Assigning: "new" = storage returned from "calloc(1088UL, 1UL)".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:377: cond_false: Condition "!new", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:377: cond_true: Condition "!nnew", taking true branch
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:378: leaked_storage: Variable "new" going out of scope leaks the storage it points to.
>
>Error: RESOURCE_LEAK (CWE-404):
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:376: alloc_fn: Storage is returned from allocation function "calloc(size_t, size_t)".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:376: var_assign: Assigning: "nnew" = storage returned from "calloc(1088UL, 1UL)".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:377: cond_true: Condition "!new", taking true branch
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:378: leaked_storage: Variable "nnew" going out of scope leaks the storage it points to.
>
>Error: TAINTED_SCALAR (CWE-20):
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:113: cond_false: Condition "entry", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:117: if_end: End of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:120: cond_false: Condition "entry", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:124: if_end: End of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:127: cond_false: Condition "!new", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:128: if_end: End of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:139: cond_false: Condition "!fd", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:142: if_end: End of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:144: tainted_string_argument: Call to "fscanf(fd, "%x", &class)" taints "class".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:147: cond_false: Condition "!rc", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:148: if_end: End of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:155: cond_false: Condition "class >= 18", taking false branch
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:156: if_end: End of if statement
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:155: upper_bounds: Checking upper bounds of signed scalar "class" by "class >= 18".
>/builddir/build/BUILD/irqbalance-1.0.4/classify.c:158: tainted_data: Using tainted variable "class" as an index into an array "class_codes".

Actions: View

Attachments on bug 878001: 647713