TE
科技回声
首页24小时热榜最新最佳问答展示工作
GitHubTwitter
首页

科技回声

基于 Next.js 构建的科技新闻平台,提供全球科技新闻和讨论内容。

GitHubTwitter

首页

首页最新最佳问答展示工作

资源链接

HackerNews API原版 HackerNewsNext.js

© 2025 科技回声. 版权所有。

A reminder to be cautious merging pull requests on NYE

7 点作者 everettForth超过 10 年前

1 comment

mc_hammer超过 10 年前
Im not saying its even 50% true but the original comitters recent commits look almost-suspect too.<p>a few things that look guilty even though they might not be:<p>(just looking at 5 of his last 1 mo of commits)<p>1) hes touching a lot of crypto 2) hes removing crypto 3) hes adding ifdefs disabling services (ie #ifdef windowsstuffflag skip_eliptic_curve_encryption ) - real example . wtf. 4) im not seeing a lot of feature additions 5) oh openssl 1.0 came out i better make 1.01 2 days later with like 2 features, my tls heartbeat code and some lame other feature 6) his heartbleed commit uses the var &quot;payload&quot; instead of &quot;data&quot; or &quot;msg&quot; - wtf.<p>7) hes reversing flags in code<p><pre><code> if (&quot;OK&quot; == vt_asdf_ok){ if (checkflags) { dostuff() </code></pre> becomes<p><pre><code> if (&quot;ok&quot; != vt_Asdf_ok) { } if (!checkflags) { &#x2F;&#x2F;dostuff </code></pre> idk i only looked at 5 commits