diff options
Diffstat (limited to 'fs/cachefiles/daemon.c')
-rw-r--r-- | fs/cachefiles/daemon.c | 38 |
1 files changed, 17 insertions, 21 deletions
diff --git a/fs/cachefiles/daemon.c b/fs/cachefiles/daemon.c index c2413561ea75..727caedcdd92 100644 --- a/fs/cachefiles/daemon.c +++ b/fs/cachefiles/daemon.c | |||
@@ -552,8 +552,7 @@ static int cachefiles_daemon_tag(struct cachefiles_cache *cache, char *args) | |||
552 | */ | 552 | */ |
553 | static int cachefiles_daemon_cull(struct cachefiles_cache *cache, char *args) | 553 | static int cachefiles_daemon_cull(struct cachefiles_cache *cache, char *args) |
554 | { | 554 | { |
555 | struct fs_struct *fs; | 555 | struct path path; |
556 | struct dentry *dir; | ||
557 | const struct cred *saved_cred; | 556 | const struct cred *saved_cred; |
558 | int ret; | 557 | int ret; |
559 | 558 | ||
@@ -573,24 +572,21 @@ static int cachefiles_daemon_cull(struct cachefiles_cache *cache, char *args) | |||
573 | } | 572 | } |
574 | 573 | ||
575 | /* extract the directory dentry from the cwd */ | 574 | /* extract the directory dentry from the cwd */ |
576 | fs = current->fs; | 575 | get_fs_pwd(current->fs, &path); |
577 | read_lock(&fs->lock); | ||
578 | dir = dget(fs->pwd.dentry); | ||
579 | read_unlock(&fs->lock); | ||
580 | 576 | ||
581 | if (!S_ISDIR(dir->d_inode->i_mode)) | 577 | if (!S_ISDIR(path.dentry->d_inode->i_mode)) |
582 | goto notdir; | 578 | goto notdir; |
583 | 579 | ||
584 | cachefiles_begin_secure(cache, &saved_cred); | 580 | cachefiles_begin_secure(cache, &saved_cred); |
585 | ret = cachefiles_cull(cache, dir, args); | 581 | ret = cachefiles_cull(cache, path.dentry, args); |
586 | cachefiles_end_secure(cache, saved_cred); | 582 | cachefiles_end_secure(cache, saved_cred); |
587 | 583 | ||
588 | dput(dir); | 584 | path_put(&path); |
589 | _leave(" = %d", ret); | 585 | _leave(" = %d", ret); |
590 | return ret; | 586 | return ret; |
591 | 587 | ||
592 | notdir: | 588 | notdir: |
593 | dput(dir); | 589 | path_put(&path); |
594 | kerror("cull command requires dirfd to be a directory"); | 590 | kerror("cull command requires dirfd to be a directory"); |
595 | return -ENOTDIR; | 591 | return -ENOTDIR; |
596 | 592 | ||
@@ -628,8 +624,7 @@ inval: | |||
628 | */ | 624 | */ |
629 | static int cachefiles_daemon_inuse(struct cachefiles_cache *cache, char *args) | 625 | static int cachefiles_daemon_inuse(struct cachefiles_cache *cache, char *args) |
630 | { | 626 | { |
631 | struct fs_struct *fs; | 627 | struct path path; |
632 | struct dentry *dir; | ||
633 | const struct cred *saved_cred; | 628 | const struct cred *saved_cred; |
634 | int ret; | 629 | int ret; |
635 | 630 | ||
@@ -649,24 +644,21 @@ static int cachefiles_daemon_inuse(struct cachefiles_cache *cache, char *args) | |||
649 | } | 644 | } |
650 | 645 | ||
651 | /* extract the directory dentry from the cwd */ | 646 | /* extract the directory dentry from the cwd */ |
652 | fs = current->fs; | 647 | get_fs_pwd(current->fs, &path); |
653 | read_lock(&fs->lock); | ||
654 | dir = dget(fs->pwd.dentry); | ||
655 | read_unlock(&fs->lock); | ||
656 | 648 | ||
657 | if (!S_ISDIR(dir->d_inode->i_mode)) | 649 | if (!S_ISDIR(path.dentry->d_inode->i_mode)) |
658 | goto notdir; | 650 | goto notdir; |
659 | 651 | ||
660 | cachefiles_begin_secure(cache, &saved_cred); | 652 | cachefiles_begin_secure(cache, &saved_cred); |
661 | ret = cachefiles_check_in_use(cache, dir, args); | 653 | ret = cachefiles_check_in_use(cache, path.dentry, args); |
662 | cachefiles_end_secure(cache, saved_cred); | 654 | cachefiles_end_secure(cache, saved_cred); |
663 | 655 | ||
664 | dput(dir); | 656 | path_put(&path); |
665 | //_leave(" = %d", ret); | 657 | //_leave(" = %d", ret); |
666 | return ret; | 658 | return ret; |
667 | 659 | ||
668 | notdir: | 660 | notdir: |
669 | dput(dir); | 661 | path_put(&path); |
670 | kerror("inuse command requires dirfd to be a directory"); | 662 | kerror("inuse command requires dirfd to be a directory"); |
671 | return -ENOTDIR; | 663 | return -ENOTDIR; |
672 | 664 | ||
@@ -683,6 +675,10 @@ int cachefiles_has_space(struct cachefiles_cache *cache, | |||
683 | unsigned fnr, unsigned bnr) | 675 | unsigned fnr, unsigned bnr) |
684 | { | 676 | { |
685 | struct kstatfs stats; | 677 | struct kstatfs stats; |
678 | struct path path = { | ||
679 | .mnt = cache->mnt, | ||
680 | .dentry = cache->mnt->mnt_root, | ||
681 | }; | ||
686 | int ret; | 682 | int ret; |
687 | 683 | ||
688 | //_enter("{%llu,%llu,%llu,%llu,%llu,%llu},%u,%u", | 684 | //_enter("{%llu,%llu,%llu,%llu,%llu,%llu},%u,%u", |
@@ -697,7 +693,7 @@ int cachefiles_has_space(struct cachefiles_cache *cache, | |||
697 | /* find out how many pages of blockdev are available */ | 693 | /* find out how many pages of blockdev are available */ |
698 | memset(&stats, 0, sizeof(stats)); | 694 | memset(&stats, 0, sizeof(stats)); |
699 | 695 | ||
700 | ret = vfs_statfs(cache->mnt->mnt_root, &stats); | 696 | ret = vfs_statfs(&path, &stats); |
701 | if (ret < 0) { | 697 | if (ret < 0) { |
702 | if (ret == -EIO) | 698 | if (ret == -EIO) |
703 | cachefiles_io_error(cache, "statfs failed"); | 699 | cachefiles_io_error(cache, "statfs failed"); |