diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2015-11-25 15:09:40 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2015-11-26 12:23:31 +0000 |
| commit | b4b35f65e24f6665bd8dd96d341758bb15b24ada (patch) | |
| tree | ec73942a4faf380200d32016e732a37a23d28169 /jurt | |
| parent | 3b3ef6aba2f6a8b27bda420f60859ccecf15a970 (diff) | |
coverity#1340230 Dereference before null check
ah!, the original code had a type in it. That's
why it was refactored to remove the null check.
Now a proper fix for cids: 1326180<->1326190
Change-Id: Iba7fd47c03eb5c157f878e0e297e8688f20ae348
(cherry picked from commit 8244fc2655e37f178f32d63133edf08def8f62c8)
Reviewed-on: https://gerrit.libreoffice.org/20179
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'jurt')
| -rw-r--r-- | jurt/com/sun/star/uno/AnyConverter.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/jurt/com/sun/star/uno/AnyConverter.java b/jurt/com/sun/star/uno/AnyConverter.java index 61c8c7e95180..94542e6e9885 100644 --- a/jurt/com/sun/star/uno/AnyConverter.java +++ b/jurt/com/sun/star/uno/AnyConverter.java @@ -621,7 +621,7 @@ public class AnyConverter break; case TypeClass.ENUM_value: if (tc == TypeClass.ENUM_value && - (null == destTClass || destType.equals( type ) /* optional destType */)) + (null == destType || destType.equals( type ) /* optional destType */)) { return object; } |