#4 La función hace algo así: static inline bool overflow_usub(unsigned int a, unsigned int b, unsigned int *res)
Y realmetne las dos instrucciones que has puesto no son equivalentes, ya que el overflow_usub también hace la asignación que viene a continuación: mtu -= hlen + sizeof(struct frag_hdr);
Por cierto, bajo mi criterio, Linus miente cuando dice que son el mismo número de líneas.
#7Y realmetne las dos instrucciones que has puesto no son equivalentes
Claro que no. Es que la segunda instrucción (que es la que provoca la bronca) supuestamente viene a sustituir a la primera en el nuevo código del kernel.
La expresión que Linus propone como sustitución, en lugar de user "overflow_usub", es esta:
if (mtu < hlen + sizeof(struct frag_hdr) + 8 )
goto fail_toobig;
mtu -= hlen + sizeof(struct frag_hdr);
Lo cual, además, sí tiene el mismo número de líneas.
Linus Torvalds te diría que your comment is shit y que hagas el fucking favor de leerte el goddammit enlace before commenting. Pero aquí en Menéame no somos como él
juajaua que animalote, pobre programador, mañana ya esta cambiandose el nick si fue suficientemente inteligente para no usar el real o de nombre si no.
Comentarios
Parece que tiene razón
Esto lo entiende todo el mundo
if (mtu < hlen + sizeof(struct frag_hdr) )
y esto NO
if (overflow_usub(mtu, hlen + sizeof(struct frag_hdr), &mtu) || mtu
#4 Linus siempre suele tenerla en estas cosas.
La función esa debe ser la referida textualmente como "crazy helper functions that nobody knows what they do"
#4 La función hace algo así:
static inline bool overflow_usub(unsigned int a, unsigned int b, unsigned int *res)
Y realmetne las dos instrucciones que has puesto no son equivalentes, ya que el overflow_usub también hace la asignación que viene a continuación:
mtu -= hlen + sizeof(struct frag_hdr);
Por cierto, bajo mi criterio, Linus miente cuando dice que son el mismo número de líneas.
El código está sacado de , como no, stackoverflow: http://stackoverflow.com/questions/33517350/what-is-overflow-usub
#7 Y realmetne las dos instrucciones que has puesto no son equivalentes
Claro que no. Es que la segunda instrucción (que es la que provoca la bronca) supuestamente viene a sustituir a la primera en el nuevo código del kernel.
La expresión que Linus propone como sustitución, en lugar de user "overflow_usub", es esta:
if (mtu < hlen + sizeof(struct frag_hdr) + 8 )
goto fail_toobig;
mtu -= hlen + sizeof(struct frag_hdr);
Lo cual, además, sí tiene el mismo número de líneas.
Linus Torvalds te diría que your comment is shit y que hagas el fucking favor de leerte el goddammit enlace before commenting. Pero aquí en Menéame no somos como él
Linus tiene razón.
Menuda noticia... Lo raro sería que lo alabase.
juajaua que animalote, pobre programador, mañana ya esta cambiandose el nick si fue suficientemente inteligente para no usar el real o de nombre si no.
Hoy hay que perdonar al pobre Linus, seguro que anda liado con el tema de las acusaciones por acoso sexual.
Feministas intentan hacer una encerrona para incriminar a Linus Torvalds por acoso sexual
Feministas intentan hacer una encerrona para incri...
breitbart.comYa me he leído el enlace antes de comentar.