[cmake-developers] [CMake 0013258]: valgrind reports memory problems in cmIfCommand.cxx

[Mantis Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
http://public.kitware.com/Bug/view.php?id=13258 
====================================================================== 
Reported By:                Clinton Stimpson
Assigned To:                
====================================================================== 
Project:                    CMake
Issue ID:                   13258
Category:                   CMake
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
====================================================================== 
Date Submitted:             2012-05-30 11:29 EDT
Last Modified:              2012-05-30 11:29 EDT
====================================================================== 
Summary:                    valgrind reports memory problems in cmIfCommand.cxx
Description: 
Using DeployQt4.cmake, I can get some odd behavior.

Steps to Reproduce: 
find_package(Qt4)
include(${QT_USE_FILE})
add_executable(myapp myapp.cpp)
target_link_libraries(myapp ${QT_LIBRARIES})
install(TARGETS myapp DESTINATION .)
include(DeployQt4)
install_qt4_executable(myapp "qtiff;qico")

Run cmake.  No build or install necessary to see the problem.

Additional Information: 
Valgrind reports:

==30955== Invalid read of size 8
==30955==    at 0x55DA147: __GI___strncasecmp_l (strcmp.S:215)
==30955==    by 0x5590EDA: ____strtod_l_internal (strtod_l.c:566)
==30955==    by 0x711377: (anonymous namespace)::GetBooleanValue(std::string&,
cmMakefile*) (cmIfCommand.cxx:242)
....

That's triggered by the if(${executable_path}) at DeployQt4.cmake line 101, and
it appears that executable_path is populated with an unrelated variable.

====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2012-05-30 11:29 Clinton StimpsonNew Issue                                    
======================================================================