Actually I've tweaked this fix, to
a/ use estrdup() - bail if out of memory.
b/ give a parse warning if some variable ref can't be resolved 
(I haven't worked out how that might happen...) and 
c/ add a boolean to ParseDoSrc to indicate that '$'s should be resolved 
(by calling ParseDoSpecialSrc - who obviously passes FALSE to ParseDoSrc).

The idea is the same though.
--sjg