diff options
| author | das <das@FreeBSD.org> | 2005-03-18 21:22:48 +0000 |
|---|---|---|
| committer | das <das@FreeBSD.org> | 2005-03-18 21:22:48 +0000 |
| commit | fb7ab3402211ae6153f97eca35aa6df1e4b75af9 (patch) | |
| tree | 06acf2a4485146f6793948db024b0bb740680d45 /sys/vm/swap_pager.c | |
| parent | c2663b2f6d80f92e1efd71257bc56c1d7f4c6b50 (diff) | |
| download | FreeBSD-src-fb7ab3402211ae6153f97eca35aa6df1e4b75af9.zip FreeBSD-src-fb7ab3402211ae6153f97eca35aa6df1e4b75af9.tar.gz | |
Move the swap_zone == NULL check earlier (i.e. before we dereference
the pointer.)
Found by: Coverity Prevent analysis tool
Diffstat (limited to 'sys/vm/swap_pager.c')
| -rw-r--r-- | sys/vm/swap_pager.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/vm/swap_pager.c b/sys/vm/swap_pager.c index 2f229de..eca68b6 100644 --- a/sys/vm/swap_pager.c +++ b/sys/vm/swap_pager.c @@ -419,6 +419,8 @@ swap_pager_swap_init(void) n2 = n; swap_zone = uma_zcreate("SWAPMETA", sizeof(struct swblock), NULL, NULL, NULL, NULL, UMA_ALIGN_PTR, UMA_ZONE_NOFREE | UMA_ZONE_VM); + if (swap_zone == NULL) + panic("failed to create swap_zone."); do { if (uma_zone_set_obj(swap_zone, &swap_zone_obj, n)) break; @@ -428,8 +430,6 @@ swap_pager_swap_init(void) */ n -= ((n + 2) / 3); } while (n > 0); - if (swap_zone == NULL) - panic("failed to create swap_zone."); if (n2 != n) printf("Swap zone entries reduced from %d to %d.\n", n2, n); n2 = n; |
