diff options
| author | Susan LoVerso <sue@wiredtiger.com> | 2015-02-09 10:01:25 -0500 |
|---|---|---|
| committer | Susan LoVerso <sue@wiredtiger.com> | 2015-02-09 10:01:25 -0500 |
| commit | 9871b1863a57fa3e35729d5e7a91faa319beba96 (patch) | |
| tree | cb9dad1f73f5239108e9b881dc17410e939a0892 | |
| parent | 21caa51b27ef16d8a19b125db021595ff83ced9d (diff) | |
| download | mongo-9871b1863a57fa3e35729d5e7a91faa319beba96.tar.gz | |
Coverity 1268396: Null pointer dereferences (REVERSE_INULL)
| -rw-r--r-- | src/os_posix/os_open.c | 2 | ||||
| -rw-r--r-- | src/os_win/os_open.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/os_posix/os_open.c b/src/os_posix/os_open.c index 5f6f98d6ecc..8e2175ee77d 100644 --- a/src/os_posix/os_open.c +++ b/src/os_posix/os_open.c @@ -214,7 +214,6 @@ __wt_close(WT_SESSION_IMPL *session, WT_FH *fh) uint64_t bucket; conn = S2C(session); - bucket = fh->name_hash % WT_HASH_ARRAY_SIZE; __wt_spin_lock(session, &conn->fh_lock); if (fh == NULL || fh->ref == 0 || --fh->ref > 0) { @@ -223,6 +222,7 @@ __wt_close(WT_SESSION_IMPL *session, WT_FH *fh) } /* Remove from the list. */ + bucket = fh->name_hash % WT_HASH_ARRAY_SIZE; WT_CONN_FILE_REMOVE(conn, fh, bucket); WT_STAT_FAST_CONN_DECR(session, file_open); diff --git a/src/os_win/os_open.c b/src/os_win/os_open.c index 8bf78aaab35..aab6359997e 100644 --- a/src/os_win/os_open.c +++ b/src/os_win/os_open.c @@ -201,7 +201,6 @@ __wt_close(WT_SESSION_IMPL *session, WT_FH *fh) uint64_t bucket; conn = S2C(session); - bucket = fh->name_hash % WT_HASH_ARRAY_SIZE; __wt_spin_lock(session, &conn->fh_lock); if (fh == NULL || fh->ref == 0 || --fh->ref > 0) { @@ -210,6 +209,7 @@ __wt_close(WT_SESSION_IMPL *session, WT_FH *fh) } /* Remove from the list. */ + bucket = fh->name_hash % WT_HASH_ARRAY_SIZE; WT_CONN_FILE_REMOVE(conn, fh, bucket); WT_STAT_FAST_CONN_DECR(session, file_open); |