[pgpool-hackers: 3743] Re: problem when getting function name

[Tatsuo Ishii]

>> It is possible that Coverity was wrong (false positive) but can you please
>> take a look at this?
> 
> 
> Since SplitIdentifierString function does not free rawstring, and rawstring seems will not be NULL in current code.
> So I think it will not cause Null pointer dereferences.
> 
> However, the 'if(rawstring)' is a redundant condition.
> we can remove it and may be Coverity will not complain again.

Thank you for prompt response! I am going to apply your patch. Let's
see what next Coverity run will say...

Best regards,
--
Tatsuo Ishii
SRA OSS, Inc. Japan
English: http://www.sraoss.co.jp/index_en.php
Japanese:http://www.sraoss.co.jp