diff options
| author | Laxmikant Rashinkar <LK.Rashinkar@gmail.com> | 2014-07-20 16:36:33 -0700 |
|---|---|---|
| committer | Laxmikant Rashinkar <LK.Rashinkar@gmail.com> | 2014-07-20 16:36:33 -0700 |
| commit | 0c63a8feb3c52de98a5da51a0a0f743450c34645 (patch) | |
| tree | cccc68684d5cac4795173a3aaee905de172f8b2f /sesman/chansrv/devredir.c | |
| parent | a990287c4627ff6b0d3b6a42b06d1f4ce242d918 (diff) | |
| download | xrdp-proprietary-0c63a8feb3c52de98a5da51a0a0f743450c34645.tar.gz xrdp-proprietary-0c63a8feb3c52de98a5da51a0a0f743450c34645.zip | |
coverity: Dereference after null check
Diffstat (limited to 'sesman/chansrv/devredir.c')
| -rw-r--r-- | sesman/chansrv/devredir.c | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/sesman/chansrv/devredir.c b/sesman/chansrv/devredir.c index 9b746a98..121c1190 100644 --- a/sesman/chansrv/devredir.c +++ b/sesman/chansrv/devredir.c @@ -790,10 +790,14 @@ dev_redir_proc_device_iocompletion(struct stream *s) fuse_data = devredir_fuse_data_dequeue(irp); if (fuse_data == NULL) + { log_error("fuse_data is NULL"); - - xfuse_devredir_cb_read_file(fuse_data->data_ptr, s->p, Length); - devredir_irp_delete(irp); + } + else + { + xfuse_devredir_cb_read_file(fuse_data->data_ptr, s->p, Length); + devredir_irp_delete(irp); + } break; case CID_WRITE: @@ -802,10 +806,14 @@ dev_redir_proc_device_iocompletion(struct stream *s) fuse_data = devredir_fuse_data_dequeue(irp); if (fuse_data == NULL) + { log_error("fuse_data is NULL"); - - xfuse_devredir_cb_write_file(fuse_data->data_ptr, s->p, Length); - devredir_irp_delete(irp); + } + else + { + xfuse_devredir_cb_write_file(fuse_data->data_ptr, s->p, Length); + devredir_irp_delete(irp); + } break; case CID_CLOSE: |