View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0013634

ParaView

(No Category)

public

2012-11-08 13:59

2013-01-30 10:41

Reporter

Sven Buijssen

Assigned To

Priority

normal

Severity

minor

Reproducibility

have not tried

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

Fixed in Version

3.98.1

Summary

0013634: [git/master] VisItBridge GMVReader's gmvread.c: stack smashing while parsing file with polygonal data

Description

Take the attached sample ASCII GMV file "gmvtest.inp" that traditionally shipped with GMV binaries since February 2000, append file extension ".gmv".

ParaView 3.98.0-RC1, Debug build for Windows 64bit with VS2008

Load "gmvtest.inp.gmv" by means of the VisItBridge GMVReader.

=> VS2008 raises dialog "Run-Time Check Failure #2 - Stack around the variable 'varname' was corrupted.", at the end of readpolygons() in gmvread.c.

Reason behind is character array "varname" has fixed length 5, while the keyword that ends polygonal data definition, "endpoly" (that is read into "varname" via scanf()), has length 7+1.

Tags

No tags attached.

Project

TBD

Topic Name

Type

incorrect functionality

Attached Files

gmvtest.inp [^] (1,854 bytes) 2012-11-08 13:59

Relationships

Notes
(0029820) Sven Buijssen (reporter) 2012-11-30 13:59	commit 243ac30705059dbec0d9e7e04e660cd12256d5d3 Author: Sven Buijssen <sven.buijssen@tu-dortmund.de> Date: Thu Nov 8 20:00:48 2012 +0100 BUG 0013634. Fixed stash smashing while reading ASCII file w/polygonal data

Issue History
Date Modified	Username	Field	Change
2012-11-08 13:59	Sven Buijssen	New Issue
2012-11-08 13:59	Sven Buijssen	File Added: gmvtest.inp
2012-11-30 13:59	Sven Buijssen	Note Added: 0029820
2012-11-30 13:59	Sven Buijssen	Status	backlog => closed
2012-11-30 13:59	Sven Buijssen	Resolution	open => fixed
2012-11-30 13:59	Sven Buijssen	Fixed in Version	=> git-master
2013-01-30 10:41	Utkarsh Ayachit	Fixed in Version	git-master => 3.98.1