[Webkit-unassigned] [Bug 169027] New: [GTK] Fix problems found by Coverity scan in WebPrintOperationGtk.cpp

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 1 03:37:09 PST 2017


https://bugs.webkit.org/show_bug.cgi?id=169027

            Bug ID: 169027
           Summary: [GTK] Fix problems found by Coverity scan in
                    WebPrintOperationGtk.cpp
    Classification: Unclassified
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit Gtk
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: tpopela at redhat.com
                CC: bugs-noreply at webkitgtk.org, cgarcia at igalia.com

65. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:365: member_decl: Class member declaration for "totalToPrint".
66. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:306: uninit_member: Non-static class member "totalToPrint" is not initialized in this constructor nor in any functions that it calls.
67. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:372: member_decl: Class member declaration for "lastPagePosition".
68. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:306: uninit_member: Non-static class member "lastPagePosition" is not initialized in this constructor nor in any functions that it calls.
#   304|           firstPagePosition = printOperation->pagePosition();
#   305|           firstSheetNumber = sheetNumber;
#   306|->     }
#   307|   
#   308|       size_t collatedCopiesLeft()

and

69. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:365: member_decl: Class member declaration for "totalToPrint".
70. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:299: uninit_member: Non-static class member "totalToPrint" is not initialized in this constructor nor in any functions that it calls.
#   297|           if (sheetNumber * numberUp >= pages.size()) {
#   298|               isValid = false;
#   299|->             return;
#   300|           }
#   301|   

actually the totalToPrint member is not used at all -> remove it.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170301/3098c5c4/attachment-0001.html>


More information about the webkit-unassigned mailing list