From 5059d796ea951d06cebc55f1a9158a8dd001af0e Mon Sep 17 00:00:00 2001 From: albert-github Date: Tue, 2 Jul 2019 15:31:41 +0200 Subject: High consequence coverity mesages - initializing fd (analogous to htmldocvisitor) - always good to close file pointers --- src/docbookvisitor.cpp | 2 +- src/dotrunner.cpp | 2 ++ src/latexdocvisitor.cpp | 2 +- src/mandocvisitor.cpp | 2 +- src/rtfdocvisitor.cpp | 2 +- src/xmldocvisitor.cpp | 2 +- 6 files changed, 7 insertions(+), 5 deletions(-) diff --git a/src/docbookvisitor.cpp b/src/docbookvisitor.cpp index 73db25f..51ddfe7 100644 --- a/src/docbookvisitor.cpp +++ b/src/docbookvisitor.cpp @@ -480,7 +480,7 @@ DB_VIS_C popEnabled(); if (!m_hide) { - FileDef *fd; + FileDef *fd = 0; if (!op->includeFileName().isEmpty()) { QFileInfo cfi( op->includeFileName() ); diff --git a/src/dotrunner.cpp b/src/dotrunner.cpp index b7ddda1..22a0081 100644 --- a/src/dotrunner.cpp +++ b/src/dotrunner.cpp @@ -128,6 +128,7 @@ bool DotRunner::readBoundingBox(const char *fileName,int *width,int *height,bool if (p) // found PageBoundingBox or /MediaBox string { int x,y; + fclose(f); if (sscanf(p+bblen,"%d %d %d %d",&x,&y,width,height)!=4) { //printf("readBoundingBox sscanf fail\n"); @@ -137,6 +138,7 @@ bool DotRunner::readBoundingBox(const char *fileName,int *width,int *height,bool } } err("Failed to extract bounding box from generated diagram file %s\n",fileName); + fclose(f); return FALSE; } diff --git a/src/latexdocvisitor.cpp b/src/latexdocvisitor.cpp index f960333..98b67a8 100644 --- a/src/latexdocvisitor.cpp +++ b/src/latexdocvisitor.cpp @@ -574,7 +574,7 @@ void LatexDocVisitor::visit(DocIncOperator *op) popEnabled(); if (!m_hide) { - FileDef *fd; + FileDef *fd = 0; if (!op->includeFileName().isEmpty()) { QFileInfo cfi( op->includeFileName() ); diff --git a/src/mandocvisitor.cpp b/src/mandocvisitor.cpp index e407642..8665746 100644 --- a/src/mandocvisitor.cpp +++ b/src/mandocvisitor.cpp @@ -388,7 +388,7 @@ void ManDocVisitor::visit(DocIncOperator *op) popEnabled(); if (!m_hide) { - FileDef *fd; + FileDef *fd = 0; if (!op->includeFileName().isEmpty()) { QFileInfo cfi( op->includeFileName() ); diff --git a/src/rtfdocvisitor.cpp b/src/rtfdocvisitor.cpp index 040375d..e4a9eba 100644 --- a/src/rtfdocvisitor.cpp +++ b/src/rtfdocvisitor.cpp @@ -551,7 +551,7 @@ void RTFDocVisitor::visit(DocIncOperator *op) popEnabled(); if (!m_hide) { - FileDef *fd; + FileDef *fd = 0; if (!op->includeFileName().isEmpty()) { QFileInfo cfi( op->includeFileName() ); diff --git a/src/xmldocvisitor.cpp b/src/xmldocvisitor.cpp index 0f11315..63949f9 100644 --- a/src/xmldocvisitor.cpp +++ b/src/xmldocvisitor.cpp @@ -423,7 +423,7 @@ void XmlDocVisitor::visit(DocIncOperator *op) popEnabled(); if (!m_hide) { - FileDef *fd; + FileDef *fd = 0; if (!op->includeFileName().isEmpty()) { QFileInfo cfi( op->includeFileName() ); -- cgit v0.12