Bug 750197

Summary: Core Dump error and close application
Product: Red Hat Enterprise Linux 6 Reporter: prashant ingale <pingale>
Component: kipi-pluginsAssignee: Lukáš Tinkl <ltinkl>
Status: CLOSED WONTFIX QA Contact: Desktop QE <desktop-qa-list>
Severity: medium Docs Contact:
Priority: medium    
Version: 6.3CC: bhubbard, jgrulich, jwest, pknirsch, than, vgaikwad
Target Milestone: rc   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-07-13 20:51:09 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 782183, 836160, 840699    
Attachments:
Description Flags
First alternative patch using isInitialized member variable
none
Second alternative patch using existing firstImage member variable none

Description prashant ingale 2011-10-31 10:29:53 UTC
Description of problem:
OpenGL image viewer, the system gives a Core Dump error and close application.

Version-Release number of selected component (if applicable):

Red Hat Enterprise Linux Workstation release 6.0 (Santiago)
architecture: x86_64


How reproducible:


Steps to Reproduce:
1.Go to Plug-in  
2.then tools
3.OpenGL image viewer
  
Actual results:


Expected results:


Additional info:

Comment 6 Brad Hubbard 2012-01-04 22:14:50 UTC
Created attachment 550771 [details]
First alternative patch using isInitialized member variable

Comment 7 Brad Hubbard 2012-01-04 22:15:45 UTC
Created attachment 550772 [details]
Second alternative patch using existing firstImage member variable

Comment 8 Brad Hubbard 2012-01-04 22:27:08 UTC
I have created two possible solutions to this issue.

That problem is caused by the fact that when the menu item is clicked with no image open a ViewerWidget is instantiated but ViewerWidget::initializeGL() never gets called and the tex member variable (GLuint[3]) is not initialized properly. When the destructor is invoked it calls glDeleteTextures(1,tex) and crashes when accessing the invalid tex. My first patch uses a dedicated variable (isInitialized) to represent the initialized state and is more intuitive IMHO. The second approach is more economical and uses an existing member variable (firstImage) to accomplish the same. This required moving some of the code in the constructor that should have no side effects but this approach is less intuitive and maybe prone to the problems of using one mechanism to accomplish multiple objectives?

Comment 12 RHEL Program Management 2012-07-10 06:38:57 UTC
This request was not resolved in time for the current release.
Red Hat invites you to ask your support representative to
propose this request, if still desired, for consideration in
the next release of Red Hat Enterprise Linux.

Comment 13 RHEL Program Management 2012-07-10 23:59:43 UTC
This request was erroneously removed from consideration in Red Hat Enterprise Linux 6.4, which is currently under development.  This request will be evaluated for inclusion in Red Hat Enterprise Linux 6.4.