(/home/sate/Testcases/c/cve/wireshark-1.2.0/epan/dissectors/packet-dcerpc-spoolss.c) |
| |
| 3221 | | | static int SpoolssReplyOpenPrinter_r(tvbuff_t *tvb, int offset, |
| 3222 | | | packet_info *pinfo, proto_tree *tree, |
| 3223 | | | guint8 *drep _U_) |
| 3224 | | | { |
| 3225 | | | dcerpc_info *di = (dcerpc_info *)pinfo->private_data; |
| 3226 | | | dcerpc_call_value *dcv = (dcerpc_call_value *)di->call_data; |
| 3227 | | | e_ctx_hnd policy_hnd; |
| 3228 | | | proto_item *hnd_item; |
| 3229 | | | guint32 status; |
| 3230 | | | |
| 3231 | | | |
| 3232 | | | |
| 3233 | | | offset = dissect_nt_policy_hnd( |
| 3234 | | | tvb, offset, pinfo, tree, drep, hf_hnd, &policy_hnd, &hnd_item, |
| 3235 | | | TRUE, FALSE); |
| 3236 | | | |
| 3237 | | | offset = dissect_doserror( |
| 3238 | | | tvb, offset, pinfo, tree, drep, hf_rc, &status); |
| 3239 | | | |
| 3240 | | | if( status == 0 ){ |
| 3241 | | | const char *pol_name; |
| 3242 | | | |
| 3243 | | | if (dcv->se_data){ |
| 3244 | | | pol_name = ep_strdup_printf( |
| 3245 | | | "ReplyOpenPrinter(%s)", (char *)dcv->se_data); |
Ignored Return Value
The return value of ep_strdup_printf() is never checked in the highlighted execution scenario. - If the return value can indicate an error, the error will be ignored if the highlighted code executes.
- The return value of ep_strdup_printf() is checked 97% of the time in this project. CodeSonar is configured to enforce Ignored Return Value checks for any function whose return value is checked at least 96% of the time, unless the function is used fewer than 20 times. (To modify these thresholds, use configuration file parameters RETURN_CHECKER_SAMPLE_SIZE and RETURN_CHECKER_RATIO. To exempt ep_strdup_printf() from the Ignored Return Value check, use configuration file parameter RETURN_CHECKER_IGNORED_FUNCS).
Show: All events | Only primary events |
|
| 3246 | | | } else { |
| 3247 | | | pol_name = "Unknown ReplyOpenPrinter() handle"; |
| 3248 | | | } |
| 3249 | | | if(!pinfo->fd->flags.visited){ |
Event 2:
Skipping " if". pinfo->fd->flags.visited evaluates to true.
hide
|
|
| 3250 | | | dcerpc_store_polhnd_name(&policy_hnd, pinfo, pol_name); |
| 3251 | | | } |
| 3252 | | | |
| 3253 | | | if(hnd_item) |
Event 3:
Skipping " if". hnd_item evaluates to false.
hide
|
|
| 3254 | | | proto_item_append_text(hnd_item, ": %s", pol_name); |
| 3255 | | | } |
| 3256 | | | |
| 3257 | | | return offset; |
| 3258 | | | } |
| |