[Ruby] Potential compaction issue in `rb_trilogy_connect`? about trilogy HOT 2 OPEN

Ok, so down the road, Trilogy will call strdup on these char *, so if there is a compaction issue, it's only during initialize. So you'd need GC.auto_compact = true and GC to trigger inside rb_trilogy_connect but before try_connect, which seem unlikely.

I'll double check, but it's probably a red herring.

from trilogy.

Alright, I think the only risk is the two handle_trilogy_error calls after try_connect:

    int rc = try_connect(ctx, &handshake, &connopt);
    if (rc == TRILOGY_TIMEOUT) {
        rb_raise(Trilogy_TimeoutError, "trilogy_connect_recv");
    }
    if (rc != TRILOGY_OK) {
        if (connopt.path) {
            handle_trilogy_error(ctx, rc, "trilogy_connect - unable to connect to %s", connopt.path);
        } else {
            handle_trilogy_error(ctx, rc, "trilogy_connect - unable to connect to %s:%hu", connopt.hostname,
                                 connopt.port);
        }
    }

AFAICT try_connect could alloc and trigger GC, causing connopt.path and connopt.hostname to be potentially pointing at garbage.

from trilogy.