diff options
author | Stefan Fritsch <sf@apache.org> | 2010-07-27 22:09:45 +0000 |
---|---|---|
committer | Stefan Fritsch <sf@apache.org> | 2010-07-27 22:09:45 +0000 |
commit | 443247b17081cb00b0e7f2f688889417cb45871f (patch) | |
tree | 1cb6ddc24a0845e85f254eec9056103e26b73d0b /dbd | |
parent | 473c87316e7cf23eadb438189d2932b86ac8afae (diff) | |
download | apr-443247b17081cb00b0e7f2f688889417cb45871f.tar.gz |
Fix various issues found by cppcheck
- error handling issues
- use of uninitialized data
- null pointer dereference
- unused variables
- memory/fd leaks
- broken code in threadproc/beos/proc.c
git-svn-id: https://svn.apache.org/repos/asf/apr/apr/trunk@979891 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'dbd')
-rw-r--r-- | dbd/apr_dbd_oracle.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/dbd/apr_dbd_oracle.c b/dbd/apr_dbd_oracle.c index 1de349018..df3ebcc67 100644 --- a/dbd/apr_dbd_oracle.c +++ b/dbd/apr_dbd_oracle.c @@ -1758,8 +1758,8 @@ static int dbd_oracle_end_transaction(apr_dbd_transaction_t *trans) { int ret = 1; /* no transaction is an error cond */ sword status; - apr_dbd_t *handle = trans->handle; if (trans) { + apr_dbd_t *handle = trans->handle; switch (trans->status) { case TRANS_NONE: /* No trans is an error here */ status = OCI_ERROR; |