| Message ID | 1502126560-24462-2-git-send-email-dvnp@cesar.org.br (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
Hello. On 08/07/2017 07:22 PM, Diogenes Pereira wrote: > Use IEEE802154_SCF_SECLEVEL_NONE macro defined at ieee802154.h file. > > Signed-off-by: Diogenes Pereira <dvnp@cesar.org.br> > --- > net/mac802154/llsec.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/net/mac802154/llsec.c b/net/mac802154/llsec.c > index 3c8ae3f..f51093e 100644 > --- a/net/mac802154/llsec.c > +++ b/net/mac802154/llsec.c > @@ -718,7 +718,8 @@ int mac802154_llsec_encrypt(struct mac802154_llsec *sec, struct sk_buff *skb) > if (hlen < 0 || hdr.fc.type != IEEE802154_FC_TYPE_DATA) > return -EINVAL; > > - if (!hdr.fc.security_enabled || hdr.sec.level == 0) { > + if (!hdr.fc.security_enabled || > + hdr.sec.level == IEEE802154_SCF_SECLEVEL_NONE) { Please align the start of the new line with the open parenthesis. With this change you can add my Acked-by: Stefan Schmidt <stefan@osg.samsung.com> regards Stefan Schmidt -- To unsubscribe from this list: send the line "unsubscribe linux-wpan" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hello. On 08/09/2017 01:07 PM, Diógenes Vila Nova Pereira wrote: > Hi Stefan, > > I can fix this same patch file and send it again ? Yes, please do so. regards Stefan Schmidt > Rgds > > *Saúde e Paz ! > []'s > Diogenes Pereira* > Senior Embedded Software Engineer > > *E-mail: * dvnp@cesar.org.br <mailto:dvnp@cesar.org.br> > *Celular:* +55 81 98895 0809 > *Comercial:* +55 81 3419 6706 > > <http://www.cesar.org.br/> > Antes de imprimir, pense em sua responsabilidade e compromisso com o > MEIO AMBIENTE. > > NOTICE: This transmittal and/or attachments is confidential. If you are > not the intended recipient, you are hereby notified that you have > received this transmittal in error and that any review, dissemination, > distribution or copying of this message is strictly prohibited. If you > have received this message in error, please notify us immediately and > immediately delete this message and all its attachments. Thank you. > > On Wed, Aug 9, 2017 at 7:47 AM, Stefan Schmidt <stefan@osg.samsung.com > <mailto:stefan@osg.samsung.com>> wrote: > > Hello. > > On 08/07/2017 07:22 PM, Diogenes Pereira wrote: > > Use IEEE802154_SCF_SECLEVEL_NONE macro defined at ieee802154.h file. > > Signed-off-by: Diogenes Pereira <dvnp@cesar.org.br > <mailto:dvnp@cesar.org.br>> > --- > net/mac802154/llsec.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/net/mac802154/llsec.c b/net/mac802154/llsec.c > index 3c8ae3f..f51093e 100644 > --- a/net/mac802154/llsec.c > +++ b/net/mac802154/llsec.c > @@ -718,7 +718,8 @@ int mac802154_llsec_encrypt(struct > mac802154_llsec *sec, struct sk_buff *skb) > if (hlen < 0 || hdr.fc.type != IEEE802154_FC_TYPE_DATA) > return -EINVAL; > - if (!hdr.fc.security_enabled || hdr.sec.level == 0) { > + if (!hdr.fc.security_enabled || > + hdr.sec.level == IEEE802154_SCF_SECLEVEL_NONE) { > > > Please align the start of the new line with the open parenthesis. > > With this change you can add my > > Acked-by: Stefan Schmidt <stefan@osg.samsung.com > <mailto:stefan@osg.samsung.com>> > > regards > Stefan Schmidt > > -- To unsubscribe from this list: send the line "unsubscribe linux-wpan" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/net/mac802154/llsec.c b/net/mac802154/llsec.c index 3c8ae3f..f51093e 100644 --- a/net/mac802154/llsec.c +++ b/net/mac802154/llsec.c @@ -718,7 +718,8 @@ int mac802154_llsec_encrypt(struct mac802154_llsec *sec, struct sk_buff *skb) if (hlen < 0 || hdr.fc.type != IEEE802154_FC_TYPE_DATA) return -EINVAL; - if (!hdr.fc.security_enabled || hdr.sec.level == 0) { + if (!hdr.fc.security_enabled || + hdr.sec.level == IEEE802154_SCF_SECLEVEL_NONE) { skb_push(skb, hlen); return 0; }
Use IEEE802154_SCF_SECLEVEL_NONE macro defined at ieee802154.h file. Signed-off-by: Diogenes Pereira <dvnp@cesar.org.br> --- net/mac802154/llsec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)