BUG/MINOR: quic: Wrong ->accept() error handling

Really signal the caller that ->accept() has failed if the session could not
be initialized because conn_complete_session() has failed. This is the case
if the mux could not be initialized too.
When it fails an ->accept() must returns -1 in case of resource shortage.
This commit is contained in:
Frédéric Lécaille 2021-03-11 17:06:30 +01:00 committed by Amaury Denoyelle
parent 7a668f5acb
commit e8139f3b37

View File

@ -69,8 +69,6 @@ int quic_session_accept(struct connection *cli_conn)
if (conn_complete_session(cli_conn) >= 0)
return 1;
return 0;
out_free_sess:
/* prevent call to listener_release during session_free. It will be
* done below, for all errors. */
@ -83,7 +81,7 @@ int quic_session_accept(struct connection *cli_conn)
conn_free(cli_conn);
out:
return 0;
return -1;
}
/*