V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
RenzHoly
V2EX  ›  Node.js

极光推送辣鸡

  •  
  •   RenzHoly · 2017-03-06 20:59:57 +08:00 · 4459 次点击
    这是一个创建于 2805 天前的主题,其中的信息可能已经有所发展或是发生改变。

    极光推送官方 Nodejs SDK 竟然 eval 去执行从服务器传回来的代码 还“聪明”地用括号括起来,呵呵 https://github.com/jpush/jpush-api-nodejs-client/blob/master/lib/JPush/JPush.js#L448

    15 条回复    2017-03-10 18:29:32 +08:00
    fyibmsd
        1
    fyibmsd  
       2017-03-06 23:20:26 +08:00
    这代码牛逼坏了
    bdbai
        2
    bdbai  
       2017-03-06 23:37:01 +08:00 via Android
    在那个 JSON 还没被浏览器广泛兼容的年代, eval 加括号解析 JSON 是很常见的事...
    DoraJDJ
        3
    DoraJDJ  
       2017-03-06 23:40:05 +08:00 via Android
    @bdbai 不过现在用 JSON.parse 就能解析 JSON 了,如果还在用老掉牙的 eval 会不会造成漏洞利用?
    SourceMan
        4
    SourceMan  
       2017-03-06 23:45:54 +08:00 via iPhone
    没毛病
    iamaprin
        5
    iamaprin  
       2017-03-07 00:10:21 +08:00
    已经修改了,官方动作有点快
    RenzHoly
        6
    RenzHoly  
    OP
       2017-03-07 00:13:29 +08:00
    @bdbai 但这里是 node
    qfdk
        7
    qfdk  
       2017-03-07 00:25:54 +08:00
    要是不改 n 个服务器又会背沦陷了 :)
    wangjie
        8
    wangjie  
       2017-03-07 00:33:03 +08:00 via iPhone
    已经改了 666
    xiezefan
        9
    xiezefan  
       2017-03-07 01:35:02 +08:00
    原谅提交这行代码的那位可怜的爪哇程序猿......
    coderzheng
        10
    coderzheng  
       2017-03-07 10:03:38 +08:00
    奇怪, V 站没改主题啊,为啥这个页面打开是黑色背景的。
    sobigfish
        11
    sobigfish  
       2017-03-07 12:20:09 +08:00
    jych1224
        12
    jych1224  
       2017-03-07 16:43:25 +08:00
    我想知道你是如何发现的
    bdbai
        13
    bdbai  
       2017-03-07 18:16:54 +08:00 via Android
    @DoraJDJ @RenzHoly 确实有安全问题,只是看到楼主一脸诧异的样子...
    RenzHoly
        14
    RenzHoly  
    OP
       2017-03-09 20:51:26 +08:00
    @jych1224 它们的服务器出了问题,发出来一个不完整的 json , eval 出错,导致我们的服务重启。看日志发现的
    Fontaine
        15
    Fontaine  
       2017-03-10 18:29:32 +08:00
    我想说 468 行 if (method === 'POST' || method === 'post') 你就是不会 method.toLowerCase() === 'post' 吗
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2528 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 05:56 · PVG 13:56 · LAX 21:56 · JFK 00:56
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.