Re: [HACKERS] unixODBC (again)

[Hiroshi Inoue]

Nick Gorham wrote: > > Then having sorted this out, I get a core dump, that I have traced to > CC_lookup_pg_version, the code did > > CC_lookup_pg_version(ConnectionClass *self) > { > HSTMT hstmt; > StatementClass *stmt; > RETCODE result; > char *szVersion = "0.0"; > static char *func = "CC_lookup_pg_version"; > > Then later did a > > sprintf( szVersion... ); > > This seems to be trying to write into, what the compiler is marking as > read only storage. A quick change to > You are right, it seems a misuse of char *. > CC_lookup_pg_version(ConnectionClass *self) > { > HSTMT hstmt; > StatementClass *stmt; > RETCODE result; > char szVersion[ 3 ]; > static char *func = "CC_lookup_pg_version"; > > strcpy( szVersion, "0.0" ); > szVersion[3] seems too short. I would increase the length and commit it soon. Regards, Hiroshi Inoue