cairo: Only apply options to table items, to avoid crash.
authorBen Pfaff <blp@cs.stanford.edu>
Sat, 7 Jan 2012 21:14:02 +0000 (13:14 -0800)
committerBen Pfaff <blp@cs.stanford.edu>
Sat, 7 Jan 2012 21:14:02 +0000 (13:14 -0800)
commit98db17130ea7872c01015bebe5f135e6b30ad1ad
tree9262678d41b834a1d3975aa87fc8d32b39cc3579
parentbc9098242e54d481b2c5e2cef970568ab5e953cb
cairo: Only apply options to table items, to avoid crash.

Most of the members of struct xr_rendering, including the 'xr'
member, are used only for table items.  In chart items, these
members are zeroed.  But xr_rendering_apply_options() attempted to
use the 'xr' member even for chart items, which caused a crash.

This reverts and replaces the provisional fix already applied by
John Darrington in commit ee7b576822f (Avoid crash rendering charts
in the gui interface.), which also fixed the crash but added extra
unnecessary work.

Reported-by: Nick Macdonald <nick@nickmacdonald.net>
src/output/cairo.c