[
https://issues.apache.org/jira/browse/ORC-25?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14713463#comment-14713463
]
ASF GitHub Bot commented on ORC-25:
-----------------------------------
Github user asandryh commented on a diff in the pull request:
https://github.com/apache/orc/pull/7#discussion_r37984425
--- Diff: c++/src/MemoryPool.cc ---
@@ -88,7 +88,7 @@ namespace orc {
template <class T>
void DataBuffer<T>::reserve(uint64_t newCapacity){
- if (newCapacity > currentCapacity) {
+ if (newCapacity > currentCapacity || !buf) {
--- End diff --
This is a redundant condition that will never be used. Does Coverity
complain about this?
> Clean up Coverity scans
> -----------------------
>
> Key: ORC-25
> URL: https://issues.apache.org/jira/browse/ORC-25
> Project: Orc
> Issue Type: Bug
> Reporter: Owen O'Malley
> Assignee: Owen O'Malley
> Attachments: orc-coverity.tiff
>
>
> I've scanned the ORC C++ code base and it identified some potential issues.
> Looking through them, so far they have all been harmless, but they should be
> cleaned up so that the analysis can come back clean.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
