diff options
| author | Kris De Gussem <kris.degussem@gmail.com> | 2012-01-20 20:17:56 +0000 |
|---|---|---|
| committer | Kris <Kris.De.Gussem@hotmail.com> | 2012-01-20 20:17:56 +0000 |
| commit | 6c185b74bcb67555a8476ad9973a761db543c490 (patch) | |
| tree | a14fba84fef7a6ab5db7bfb9c6bef9d92b6afd3f /src | |
| parent | fix memleak (diff) | |
| download | inkscape-6c185b74bcb67555a8476ad9973a761db543c490.tar.gz inkscape-6c185b74bcb67555a8476ad9973a761db543c490.zip | |
cppcheck
tells us 'line 154: "Possible null pointer dereference: _stream - otherwise it is redundant to check if _stream is null at line 147"
Cannot check code on linux, but when you should get here _stream should be not NULL (if I interpreted correctly)
(bzr r10915)
Diffstat (limited to 'src')
| -rw-r--r-- | src/extension/internal/latex-text-renderer.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/extension/internal/latex-text-renderer.cpp b/src/extension/internal/latex-text-renderer.cpp index fae10a0fe..12b42a78c 100644 --- a/src/extension/internal/latex-text-renderer.cpp +++ b/src/extension/internal/latex-text-renderer.cpp @@ -144,12 +144,10 @@ LaTeXTextRenderer::setTargetFile(gchar const *filename) { g_free(filename_ext); } - if (_stream) { - /* fixme: this is kinda icky */ + /* fixme: this is kinda icky */ #if !defined(_WIN32) && !defined(__WIN32__) - (void) signal(SIGPIPE, SIG_IGN); + (void) signal(SIGPIPE, SIG_IGN); #endif - } fprintf(_stream, "%%%% Creator: Inkscape %s, www.inkscape.org\n", PACKAGE_STRING); fprintf(_stream, "%%%% PDF/EPS/PS + LaTeX output extension by Johan Engelen, 2010\n"); |