(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/dissectors/packet-dcerpc-spoolss.c) |
| |
| 4974 | | | dissect_spoolss_doc_info_data(tvbuff_t *tvb, int offset, packet_info *pinfo, |
| 4975 | | | proto_tree *tree, guint8 *drep) |
| 4976 | | | { |
| 4977 | | | dcerpc_info *di = pinfo->private_data; |
| 4978 | | | guint32 info_level = 1; |
| 4979 | | | |
| 4980 | | | if (di->conformant_run) |
| 4981 | | | return offset; |
| 4982 | | | |
| 4983 | | | switch(info_level) { |
Redundant Condition
In switch(info_level), the default case will never be executed. This may be because: - There is a constant assignment to one or more of the variables involved.
- An earlier conditional statement has already ensured that this case cannot occur.
- A crashing bug occurs on every path where this case would have been executed. Look for a preceding Null Pointer Dereference or Division By Zero warning.
|
|
| 4984 | | | case 1: |
| 4985 | | | offset = dissect_spoolss_doc_info_1( |
| 4986 | | | tvb, offset, pinfo, tree, drep); |
| 4987 | | | break; |
| 4988 | | | default: |
| 4989 | | | proto_tree_add_text( |
| 4990 | | | tree, tvb, offset, 0, |
| 4991 | | | "[Unknown documentinfo level %d]", info_level); |
| 4992 | | | break; |
| 4993 | | | } |
| 4994 | | | |
| 4995 | | | return offset; |
| 4996 | | | } |
| |