7 Oct
2015
7 Oct
'15
7:16 p.m.
On 01 Oct 2015, at 15:23, Neels Hofmeyr nhofmeyr@sysmocom.de wrote:
OAP_UNINITIALIZED = 0, // just allocated.
No C99 comments
+static int oap_evaluate_challenge(const struct oap_state *state,
const uint8_t *rx_random,const uint8_t *rx_autn,uint8_t *tx_xres)+{
- OSMO_ASSERT(sizeof(auth.u.umts.k) == sizeof(state->secret_k));
- OSMO_ASSERT(sizeof(auth.u.umts.opc) == sizeof(state->secret_opc));
static assert?
- memcpy(auth.u.umts.k, state->secret_k, sizeof(auth.u.umts.k));
- memcpy(auth.u.umts.opc, state->secret_opc, sizeof(auth.u.umts.opc));
- memset(auth.u.umts.amf, 0, 2);
sizeof?
- auth.u.umts.sqn = 42; /* TODO */
In three years we wonder what the todo is. I think it is a good habbit to add a short summary what we want to do.
+struct msgb *oap_encoded(const struct oap_message *oap_msg)
encode? Sure it is encoded after the function is done but it triggers the encoding?
+{ +int oap_register(struct oap_state *state, struct msgb **msg_tx)
return the struct msgb*? If it is NULL you already know it didnt't work and the -1/0 carry as much value right now?
- case OAP_MSGT_REGISTER_RESULT:
// successfully registered!
no C99 comments (sorry to have away the method again)
I have not looked at the state machine, we can do it together.
holger