Bug Tracker Incident #11846

 


Summary
Project
INCDT - CSVimp
Incident Category
Bugs
Incident Number
11846
Visibility
public


Product Version
0.4.2-CSVimp
Fixed In Version
0.4.4-CSVimp


Summary
embedded CSVimp crashes for this csv/atlas


Description

This scenario works with the standalone CSVimp

Also, the import actually works but instead of receiving the success message the client crashes

Attached are the two CSV files and corresponding atlas files

It will likely work for the first and crash on the second

Make certain that you have set Sales Config to Auto/Allow Override for SO#


Details
Reporter
matherton
CRM Account
xTuple
Status
Closed
Assigned
gmoskowitz


Priority
High
Severity
Crash
Updated
09-06-12 15:41
Resolution
Fixed



Comments

DateUsernameComment
09/06/11 02:14nagesh

Tested with CSVImp in xtuple 3.8.0 Alpha:Issue fixed

07/29/11 15:39gmoskowitz

The crash specifically occurred when the CSV windows were closed then reopened. Since closing the windows in the stand-alone CSVImp quits the program, that program never showed the symptom.

Tested successfully on Mac and Linux.

This is fixed in the 0.4.4 release of CSVImp and whatever xTuple ERP release comes next. The change should be backward-compatible with previous releases of xTuple ERP by copying the new CSVImp plugin into position.

07/29/11 15:34svn

Revision: 12253
http://postbooks.svn.sourceforge.net/postbooks/?rev=12253&view=rev
Author: gilmoskowitz
Date: 2011-07-29 19:33:58 +0000 (Fri, 29 Jul 2011)

Log Message:
-----------
bug #11846 - a parent object can't reuse the _msghandler of a child object
since it might be freed when the child is destroyed

Modified Paths:
--------------
csvimp/trunk/plugin/csvimpplugin.cpp

05/04/11 05:57nagesh

Tested in 370:Issue still reproducible

03/04/11 08:25mfgadmin

Steps to Reproduce:

1) Run this on the demo DB (tested on 3.5.2)

2) Make certain you do not have an SO 50500 - it creates it

3) Run the first import CSV and Atlas (the header) - first line is a header line

--it may crash here

4) Do not logoff

5) Open List Open SOs and click All Sites - you should see SO 50500

6) Run the second import CSV and Atlas (the lines) - first line is header

--it should crash here

7) Logon and open List Open SOs - you should see SO 50500 which if opened will show two line items

03/04/11 08:25mfgadmin

This scenario works with the standalone CSVimp

Also, the import actually works but instead of receiving the success message the client crashes

Attached are the two CSV files and corresponding atlas files

It will likely work for the first and crash on the second

Make certain that you have set Sales Config to Auto/Allow Override for SO#

10/19/10 11:04gmoskowitz

Fix should appear in xTuple ERP 3.6.0 and xTuple Connect 3.5.1

10/19/10 10:59gmoskowitz

The app crashed intermittently on Linux when switching back and forth between the header and item files and atlases, then importing. The valgrind tool indicated an invalid memory reference even in cases when the application didn''t crash. The fix was tested with both the xtuple erp gui client and batch manager trying imports with the modified csvimp plugin under valgrind and verifying that the invalid memory reference was no longer reported.

10/19/10 10:56svn

Revision: 10481
http://postbooks.svn.sourceforge.net/postbooks/?rev=10481&view=rev
Author: gilmoskowitz
Date: 2010-10-19 14:56:08 +0000 (Tue, 19 Oct 2010)

Log Message:
-----------
bug #11846 - valgrind showed ''invalid read of size 4'' in the
csvtoolwindow during a qobject_cast<>

Modified Paths:
--------------
csvimp/trunk/plugin/csvtoolwindow.cpp

10/04/10 16:24matherton

NOTE:

This example is now on the DB mfg36b2_initial on the qa server (mars).

10/04/10 09:27matherton

I have now re-tested this on 3.6b2 on both Vista and Ubuntu. It crashes on both.

The environment for this already exists on the QA (mars) server in DB mfg36b2 so please test it there:

1) Open embedded CSVimp utility and import using the Header map (see attached)
2) There is already an SO from a previous import in the DB so you will see a DB log message
3) Close the embedded CSVimp utility but not the application client
4) Immediately reopen CSVimp and re-import the header again. This time the application client will crash (both Vista and Ubuntu)

09/22/10 18:00joesingleton

don''t know. only running 3.5.1.

09/22/10 15:24cryan

using the 3.6 series on linux I was only able to get it crash once on the first import but could never get it to crash on the second. Are you still seeing the problem with the latest release?

08/11/10 15:57joesingleton

0011866 is not a duplicate of this. please reread. totally different behavior.

08/11/10 09:40mmcbride

confirmed in 353

08/10/10 18:35joesingleton

I experienced the same problem trying to import via System > Master Information > Maintain CSV Atlases.



Characteristics

Backport
No
CodeReview
Needed
Operating System
ALL
Doc Flag
False
Copyright Assigned
Yes
HaxTuple
No
Points
None

Files

Filename
C5_1a_SO_Head.csv
C5_1a_SO_Head_Map.xm
C5_1b_SO_Item_Map.xm
C5_1b_SO_line.csv


Related Documents

TypeNumberDescriptionRelationshipRemove
Incident11866"CSV Tool" window exits xTuple Application.Related tox
Incident350Invoice #5205Parentx
Incident5260cycle count creation db functions wrongRelated tox
Incident5261Cannot delete Count TagRelated tox
Incident5262Diff % not correct in FRERelated tox
Incident5263Ship-to pattern ignores customer selectionRelated tox
ProjectCSVIMPPorted From Mantisx


Subscribers

You do not have permission to view subscribers.


Incident History

DateUsernameFieldChange
12/16/10 09:17acdrupalNewIncident Added
05/04/11 05:57nageshStatusStatus Changed: Resolved -> Assigned
05/04/11 05:57acdrupalCharacteristic Estimated Hours Added: ""
07/29/11 15:41gmoskowitzDescription Updated: "This scenario works ..." -> "This scenario works ..."
07/29/11 15:41gmoskowitzFixed In: -> 0.4.4
07/29/11 15:41gmoskowitzCharacteristic CodeReview Added: "Needed"
07/29/11 15:41gmoskowitzStatusStatus Changed: Assigned -> Resolved
09/06/11 02:14nageshDescription Updated: "This scenario works ..." -> "This scenario works ..."
09/06/11 02:14nageshStatusStatus Changed: Resolved -> Closed