ydb_file_id_free()/ydb_file_is_identical()/ydb_file_name_to_id() and their _t variants should issue PARAMINVALID error if input filename/fileid pointer is NULL
Final Release Note
ydb_file_id_free()
/ydb_file_id_free_t()
, ydb_file_is_identical()
/ydb_file_is_identical()_t
, and ydb_file_name_to_id()
/ydb_file_name_to_id_t()
issue a PARAMINVALID error in case the input file name or file id parameter is NULL. Previously, they could cause the process to terminate abnormally with a SIG-11 if the file id parameter was NULL, and returned YDB_OK in case the file name parameter was NULL. [#418 (closed)]
Description
Currently the Utility functions listed in the issue title assume the input fileid parameter is non-NULL. They could SIG-11 if one passes a NULL pointer. It is desirable to issue a PARAMINVALID error in this case just like other SimpleAPI functions too.
Draft Release Note
ydb_file_id_free()/ydb_file_is_identical()/ydb_file_name_to_id() and ydb_file_id_free_t()/ydb_file_is_identical_t()/ydb_file_name_to_id_t() now issue a PARAMINVALID error in case the input file name or file id parameter to any of these functions is NULL. Previously, they could cause the process to terminate abnormally with a SIG-11 if the file id parameter was NULL and returned YDB_OK in case the file name parameter was NULL.