App crash when run some queries.

[huyphams]

The lib would crash when we run the following queries:

1. Crash when trying to run a query with invalid quotes.

select * from ‟food‟;

2. Crash when trying to show some binary column

Create table

create table tableplus_crash_example (col AggregateFunction(argMax, Int32, DateTime(3))) engine = Memory;
insert into tableplus_crash_example values (unhex('010000000001089170A883010000'));

Then run this query.

select col from tableplus_crash_example;

[Enmk]

Hi @huyphams thanks for your report, what is the server version?

[huyphams]

I can reproduce with the latest commit 4774da1

[Enmk]

Ok, what is the server version? There was recently a breaking change on aggregated functions state serializations (I believe between 22.8 and 22.10) so I'm curious is that related...

[huyphams]

I'm testing with the ClickHouse version 21.12.3.32

[huyphams]

Hi, @Enmk any update on this? we're holding back our release due to this issue. Should we move on or wait for the patch?
Thank you!

[Enmk]

@huyphams please see #273 for test sample, does that reproduces your case?

I tested that against 21.12 localy and it works fine:

Note: Google Test filter = *Issue266*
[==========] Running 2 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 2 tests from Client/ClientCase
[ RUN      ] Client/ClientCase.Issue266/0

+-----------+
| version() |
+-----------+
|    String |
+-----------+
| 21.12.4.1 |
+-----------+



+------+
|  col |
+------+
| Void |
+------+
| Unable to print value of type Void |
+------+

Could you please provide any details regarding the crash? Maybe some stacktrace?

[huyphams]

Hi @Enmk could you check again? I saw that all the tests failed. Btw can you also the first example as it would cause the app to crash?

[Enmk]

The crash itself is fixed here: #273

However, clickhouse-cpp currently doesn't support reading values of type AggregateFunction(...), so you woudn't be able to SELECT the result. As a workaround, you might want to cast it to String or a FixedString in the SELECT statement.

[huyphams]

I'm still getting the crash with the query

select * from ‟untitled_table_5‟;