After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 676068 - Setting size on ClutterFlowLayout container crashes
Setting size on ClutterFlowLayout container crashes
Status: RESOLVED FIXED
Product: clutter
Classification: Platform
Component: general
1.10.x
Other Linux
: Normal normal
: ---
Assigned To: clutter-maint
clutter-maint
Depends on:
Blocks:
 
 
Reported: 2012-05-15 04:32 UTC by Mike Ruprecht
Modified: 2012-05-22 13:53 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Test case demonstrating this bug (1.12 KB, text/plain)
2012-05-15 04:33 UTC, Mike Ruprecht 		  Details
Patch that fixes the crash for me (763 bytes, patch)
2012-05-15 04:35 UTC, Mike Ruprecht 		committed Details | Review
Backtrace of crash with testcase (10.14 KB, text/plain)
2012-05-15 07:04 UTC, Mike Ruprecht 		  Details

Description Mike Ruprecht 2012-05-15 04:32:50 UTC 
When creating a FlowLayout container, setting a specific size on it, and adding a child to it, as per the attached testcase, it crashes. The line on the backtrace doesn't really make sense, but from looking over it, it appears that it's probably because priv->line_natural is NULL. The attached patch makes it so in this case, priv->line_natural is allocated.
Comment 1 Mike Ruprecht 2012-05-15 04:33:55 UTC 
Created attachment 214055 [details]
Test case demonstrating this bug
Comment 2 Mike Ruprecht 2012-05-15 04:35:11 UTC 
Created attachment 214056 [details] [review]
Patch that fixes the crash for me
Comment 3 Mike Ruprecht 2012-05-15 07:04:16 UTC 
Created attachment 214062 [details]
Backtrace of crash with testcase

Oops. Just noticed I missed uploading the backtrace. Here it is.
Comment 4 Emmanuele Bassi (:ebassi) 2012-05-21 12:50:44 UTC 
Review of attachment 214056 [details] [review]:

looks okay