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/chansrv_fuse.c | |
| parent | a990287c4627ff6b0d3b6a42b06d1f4ce242d918 (diff) | |
| download | xrdp-proprietary-0c63a8feb3c52de98a5da51a0a0f743450c34645.tar.gz xrdp-proprietary-0c63a8feb3c52de98a5da51a0a0f743450c34645.zip | |
coverity: Dereference after null check
Diffstat (limited to 'sesman/chansrv/chansrv_fuse.c')
| -rw-r--r-- | sesman/chansrv/chansrv_fuse.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/sesman/chansrv/chansrv_fuse.c b/sesman/chansrv/chansrv_fuse.c index 5bef552d..e80f93a9 100644 --- a/sesman/chansrv/chansrv_fuse.c +++ b/sesman/chansrv/chansrv_fuse.c @@ -1781,12 +1781,15 @@ void xfuse_devredir_cb_rename_file(void *vp, tui32 IoStatus) new_xinode = xfuse_get_inode_from_pinode_name(fip->new_inode, fip->new_name); - if (new_xinode->mode & S_IFREG) - xfuse_delete_file_with_xinode(new_xinode); - else - xfuse_delete_dir_with_xinode(new_xinode); + if (new_xinode) + { + if (new_xinode->mode & S_IFREG) + xfuse_delete_file_with_xinode(new_xinode); + else + xfuse_delete_dir_with_xinode(new_xinode); - new_xinode = NULL; + new_xinode = NULL; + } } old_xinode = xfuse_get_inode_from_pinode_name(fip->inode, fip->name); |