summaryrefslogtreecommitdiff
path: root/mach-defpager
diff options
context:
space:
mode:
authorJustus Winter <4winter@informatik.uni-hamburg.de>2014-03-12 13:10:04 +0100
committerJustus Winter <4winter@informatik.uni-hamburg.de>2014-03-12 13:10:04 +0100
commit3deb2d7685f9e9b502a992491f747c3a657e4d15 (patch)
treef069beddf2c11bb107c6239ff7c152a9656c62a0 /mach-defpager
parentf3d9cf9f788e5b98d270309c03f44b26568944eb (diff)
mach-defpager: fix warning about uninitialized variable
Previously, failure to look up the given partition was detected after the loop by checking whether the loop ran over all existing partitions. Initialize part to NULL and check for that instead. This retains the behavior, but expresses it in a way the compiler understands better. * mach-defpager/default_pager.c (destroy_paging_partition): Initialize part to NULL and check for it still being NULL after the loop.
Diffstat (limited to 'mach-defpager')
-rw-r--r--mach-defpager/default_pager.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/mach-defpager/default_pager.c b/mach-defpager/default_pager.c
index 33d72b3a..1b9d5114 100644
--- a/mach-defpager/default_pager.c
+++ b/mach-defpager/default_pager.c
@@ -1991,7 +1991,7 @@ destroy_paging_partition(name, pp_private)
void **pp_private;
{
unsigned int id = part_id(name);
- partition_t part;
+ partition_t part = NULL;
boolean_t all_ok = TRUE;
default_pager_t entry;
int pindex;
@@ -2006,7 +2006,7 @@ destroy_paging_partition(name, pp_private)
part = partition_of(pindex);
if (part && (part->id == id)) break;
}
- if (pindex == all_partitions.n_partitions) {
+ if (! part) {
pthread_mutex_unlock(&all_partitions.lock);
return KERN_INVALID_ARGUMENT;
}