<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>169027</bug_id>
          
          <creation_ts>2017-03-01 03:37:09 -0800</creation_ts>
          <short_desc>[GTK] Fix problems found by Coverity scan in WebPrintOperationGtk.cpp</short_desc>
          <delta_ts>2017-03-08 06:48:44 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>104114</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Tomas Popela">tpopela</reporter>
          <assigned_to name="Tomas Popela">tpopela</assigned_to>
          <cc>bugs-noreply</cc>
    
    <cc>cgarcia</cc>
    
    <cc>mcatanzaro</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1282228</commentid>
    <comment_count>0</comment_count>
    <who name="Tomas Popela">tpopela</who>
    <bug_when>2017-03-01 03:37:09 -0800</bug_when>
    <thetext>65. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:365: member_decl: Class member declaration for &quot;totalToPrint&quot;.
66. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:306: uninit_member: Non-static class member &quot;totalToPrint&quot; 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 &quot;lastPagePosition&quot;.
68. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:306: uninit_member: Non-static class member &quot;lastPagePosition&quot; is not initialized in this constructor nor in any functions that it calls.
#   304|           firstPagePosition = printOperation-&gt;pagePosition();
#   305|           firstSheetNumber = sheetNumber;
#   306|-&gt;     }
#   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 &quot;totalToPrint&quot;.
70. webkitgtk-2.14.5/Source/WebKit2/WebProcess/WebPage/gtk/WebPrintOperationGtk.cpp:299: uninit_member: Non-static class member &quot;totalToPrint&quot; is not initialized in this constructor nor in any functions that it calls.
#   297|           if (sheetNumber * numberUp &gt;= pages.size()) {
#   298|               isValid = false;
#   299|-&gt;             return;
#   300|           }
#   301|   

actually the totalToPrint member is not used at all -&gt; remove it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1282229</commentid>
    <comment_count>1</comment_count>
      <attachid>303058</attachid>
    <who name="Tomas Popela">tpopela</who>
    <bug_when>2017-03-01 03:41:35 -0800</bug_when>
    <thetext>Created attachment 303058
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1282251</commentid>
    <comment_count>2</comment_count>
      <attachid>303058</attachid>
    <who name="Tomas Popela">tpopela</who>
    <bug_when>2017-03-01 06:16:01 -0800</bug_when>
    <thetext>Comment on attachment 303058
Patch

Clearing flags on attachment: 303058

Committed r213217: &lt;http://trac.webkit.org/changeset/213217&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1282252</commentid>
    <comment_count>3</comment_count>
    <who name="Tomas Popela">tpopela</who>
    <bug_when>2017-03-01 06:16:10 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>303058</attachid>
            <date>2017-03-01 03:41:35 -0800</date>
            <delta_ts>2017-03-01 06:16:01 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-169027-20170301123838.patch</filename>
            <type>text/plain</type>
            <size>1618</size>
            <attacher name="Tomas Popela">tpopela</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjEzMjE2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggOThmNjljYTczNWYxZDRi
NjI2YzZmOTBjZDBiN2NlYjgzMTk0N2JmMS4uMDQ4NTFjYzJhOTZjYzgyNDRjNDYxNjM2YzYyMDQz
MTcxZjA0YzRmZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDE3LTAzLTAxICBUb21h
cyBQb3BlbGEgIDx0cG9wZWxhQHJlZGhhdC5jb20+CisKKyAgICAgICAgW0dUS10gRml4IHByb2Js
ZW1zIGZvdW5kIGJ5IENvdmVyaXR5IHNjYW4gaW4gV2ViUHJpbnRPcGVyYXRpb25HdGsuY3BwCisg
ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNjkwMjcKKwor
ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBJbml0aWFsaXpl
IHRoZSBsYXN0UGFnZVBvc2l0aW9uIHZhcmlhYmxlIGFuZCByZW1vdmUgYW4gdW51c2VkIG9uZQor
ICAgICAgICAtIHRvdGFsVG9QcmludC4KKworICAgICAgICAqIFdlYlByb2Nlc3MvV2ViUGFnZS9n
dGsvV2ViUHJpbnRPcGVyYXRpb25HdGsuY3BwOgorICAgICAgICAoV2ViS2l0OjpQcmludFBhZ2Vz
RGF0YTo6UHJpbnRQYWdlc0RhdGEpOgorCiAyMDE3LTAzLTAxICBBbmRyZWFzIEtsaW5nICA8YWts
aW5nQGFwcGxlLmNvbT4KIAogICAgICAgICBNb3ZlIE1lbW9yeVByZXNzdXJlSGFuZGxlciB0byBX
VEYKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9ndGsvV2Vi
UHJpbnRPcGVyYXRpb25HdGsuY3BwIGIvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdl
L2d0ay9XZWJQcmludE9wZXJhdGlvbkd0ay5jcHAKaW5kZXggYjYyMDVkYjk0OTUwMTNlNGE4ZTIy
MWNkNmJmYjZkODQ5ODM4MzAxMi4uZmRkNDE5Mzk0OGY3ZjZlMDVkMjEyMWZjZTFlN2I2ZjNhZjY1
YmNiNCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL2d0ay9X
ZWJQcmludE9wZXJhdGlvbkd0ay5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9X
ZWJQYWdlL2d0ay9XZWJQcmludE9wZXJhdGlvbkd0ay5jcHAKQEAgLTIxMCw2ICsyMTAsNyBAQCBz
dHJ1Y3QgUHJpbnRQYWdlc0RhdGEgewogICAgICAgICAsIGZpcnN0U2hlZXROdW1iZXIoMCkKICAg
ICAgICAgLCBudW1iZXJPZlNoZWV0cygwKQogICAgICAgICAsIGZpcnN0UGFnZVBvc2l0aW9uKDAp
CisgICAgICAgICwgbGFzdFBhZ2VQb3NpdGlvbigwKQogICAgICAgICAsIGNvbGxhdGVkKDApCiAg
ICAgICAgICwgdW5jb2xsYXRlZCgwKQogICAgICAgICAsIGlzRG9uZShmYWxzZSkKQEAgLTM2Miw3
ICszNjMsNiBAQCBzdHJ1Y3QgUHJpbnRQYWdlc0RhdGEgewogICAgIEdSZWZQdHI8R01haW5Mb29w
PiBtYWluTG9vcDsKIAogICAgIGludCB0b3RhbFByaW50ZWQ7Ci0gICAgc2l6ZV90IHRvdGFsVG9Q
cmludDsKICAgICBpbnQgcGFnZU51bWJlcjsKICAgICBWZWN0b3I8c2l6ZV90PiBwYWdlczsKICAg
ICBzaXplX3Qgc2hlZXROdW1iZXI7Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>