Submitted by: Claudio Valderrama C. (robocop)

Assigned to: Claudio Valderrama C. (robocop)

Is related to QA423

SQL> set term ^;
SQL> create procedure p returns(a int) as begin a = 9; suspend; end^
SQL> create view vp1 as select a from p^
SQL> set term ;^
SQL> show view vp1;
A INTEGER^C

(had to be killed and hence the ^C in the output).

The reason is that isql is not prepared to find view field's sources in procedures and therefore ISQL_get_base_column_null_flag (called by ISQL_get_null_flag and this by show_table) goes in an infinite loop. The code should indicate to the caller when it can't find the required field.

The code should check also procedure parameters. I continue thinking that putting the type of source field in rdb$view_relations would simplify code in the long term and we would know where to check instead of digging in tables and procedures and who knows what else more in the future.

Commits: cd28279 31848bf 1399401 094320e 22af8f1