Jeffrin Jose <[email protected]> writes: > case TIOCINQ: { > /* > * These two are safe on a single CPU system as only > * user tasks fiddle here > */ > - struct sk_buff *skb = skb_peek(&sk->sk_receive_queue); > - long amount = 0; > + struct sk_buff *skb = skb_peek(&sk->sk_receive_queue); > + long amount = 0; > > - if (skb) > - amount = skb->len - sizeof(struct ddpehdr); > + if (skb) > + amount = skb->len - sizeof(struct ddpehdr); > rc = put_user(amount, (int __user *)argp); > break; > }
Is putting "rc = put_user(amount, (int __user *)argp);" on the same indentation level as "amount = skb->len - sizeof(struct ddpehdr);" really what you want to do? Eric -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
