[riot-notifications] [RIOT-OS/RIOT] pkg/semtech-loramac: rework how session keys are get/set (#11783)

Francisco notifications at github.com
Tue Jul 16 10:44:29 CEST 2019


Looking into the code it seems to me the problem is actually here:
```
    MibRequestConfirm_t mibReq;
    mibReq.Type = MIB_APP_SKEY;
    memcpy(mibReq.Param.AppSKey, skey, LORAMAC_APPSKEY_LEN);
    LoRaMacMibSetRequestConfirm(&mibReq);
```

`mibReq.Param.AppSKey` is an uninitialized pointer, so calling memcpy here is incorrect. What should actually be done is simply:

`mibReq.Param.AppSKey = (uint8_t *) skey;`

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11783#issuecomment-511724718
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190716/036a1aa9/attachment-0001.html>


More information about the notifications mailing list