Fix segfault when column header is skipped - #3546

This will probably have a merge conflict with #3525 -- just a note in case we want to merge in a particular order.

Changelog entry nag, and this can be merged :D

Sure -- I wouldn't mind getting confirmation from @martynbristow that this works for him first, as well.

Hello

I’m just trying to check it through
I’ve just received a Python(2483,0x7fff7c18c300) malloc: *** error for object 0x100306d10: incorrect checksum for freed object - object was probably modified after being freed.
*** set a breakpoint in malloc_error_break to debug
Abort trap: 6
I’ll see if its something I have done, as I have my real satrapy here too
Martyn Bristow
Mobile Tel: (+44) 7825 165-420
About Me: http://about.me/marty.mcfly http://about.me/marty.mcfly
Webpage:- http://www.martynbristow.co.uk http://www.martynbristow.co.uk/
E-mail:- [email protected] mailto:[email protected]
E-mail:- [email protected] mailto:[email protected]

On 24 Feb 2015, at 18:41, Michael Droettboom [email protected] wrote:

Sure -- I wouldn't mind getting confirmation from @martynbristow https://github.com/martynbristow that this works for him first, as well.

—
Reply to this email directly or view it on GitHub #3546 (comment).

@mdboom - works for me, thanks! After doing this I get the expected exception (because the data file is really malformed for the tab format):

CParserError: an error occurred while parsing table data: too many columns found in line 1 of data

Yeah brilliant
All fixed, plus it now compensates for stupid text headers ;)

On 24 Feb 2015, at 18:56, Tom Aldcroft [email protected] wrote:

@mdboom https://github.com/mdboom - works for me, thanks! After doing this I get the expected exception (because the data file is really malformed for the tab format):

CParserError: an error occurred while parsing table data: too many columns found in line 1 of data
—
Reply to this email directly or view it on GitHub #3546 (comment).

As feared, this needs to be rebased now.

@mdboom Can you rebase this? If you're not around no problem though, I'll just merge by hand.

Rebased.

Oh, I think this broke as a result of the changes I made to the tests in #3525.

FWIW I fixed that up and rebased it--the new branch is testing here: https://travis-ci.org/embray/astropy/builds/53374609

Looks like it should pass, then I'll merge.

Manually merged in 00d223f.

Thanks!

Thanks both!!