This patch removes all calls to release_value in varobj.c, except for one. We only need to call release_value when we're storing a value for indefinite future use, and install_new_value is the only such place. The extra release_value calls either are time-consuming no-ops, or lead to memory leaks, depending on the value type. OK? - Volodya * varobj.c (c_value_of_root, c_value_of_child) (cplus_value_of_child): Don't call release_value.