安全矩阵

 找回密码
 立即注册
搜索
查看: 6559|回复: 0

同事写了一个update,误用一个双引号,生产数据全变0了!

[复制链接]

991

主题

1063

帖子

4315

积分

论坛元老

Rank: 8Rank: 8

积分
4315
发表于 2021-2-10 22:13:30 | 显示全部楼层 |阅读模式
本帖最后由 gclome 于 2021-2-10 22:14 编辑

原文链接:同事写了一个update,误用一个双引号,生产数据全变0了!

一、前言

最近经常碰到开发误删除误更新数据,这不,他们又给我找了个麻烦,我们来看下整个过程,把我坑得够惨。

二、过程
由于开发需要在生产环节中修复数据,需要执行120条SQL语句,需要将数据进行更新,于是开发连上了生产数据库,首先执行了第一条SQL
  1. update tablename set source_name = "bj1062-北京市朝阳区常营北辰福第"  

  2. where source_name =     "-北京市朝阳区常营北辰福第"
复制代码

我们仔细看了下,这个SQL,的确没有什么问题,where条件也是正常的,大意就是将这个地址的前面加字符串bj1062,是真的没有错误么?是的没有错误。开发执行完成后,结果的确是符合预期。

然后开发执行了剩下的SQL,都是和上面的SQL一样,将地址进行更新。执行完成后,开发懵逼了,发现source_name都变成了0,开发赶紧给我打电话说:
涛哥,我执行了update,where条件都是对的,set的值也是对的,但是set后的字段全部都变成了0,你赶紧帮我看看,看看能不能恢复数据。
我赶紧登上服务器,查看了这段时间的binlog,发现了大量的update tablename set source_name=0的语句,利用binlog2sql进行了解析,项目地址:


赶紧和开发确定了操作的时间点,生成flashback的SQL,进行了数据恢复,同时保留现场证据。

然后对开发执行的SQL进行了check,发现了几条很诡异的SQL


这几条SQL的引号位置跑到了where 字段名字后面,简化后的SQL变成了:

update tbl_name set str_col="xxx" = "yyy"

那么这个SQL在MySQL他是如何进行语义转化的呢?

可能是下面这样的么?

update tbl_name set (str_col="xxx" )= "yyy"

这样就语法错误了,那么只会是下面这样的形式,

update tbl_name set str_col=("xxx" = "yyy")


select "xxx" = "yyy"

的值是0,所以

update tbl_name set str_col="xxx" = "yyy"

等价于

update tbl_name set str_col=0

所以就导致了source_name字段全部更新成了0.

我们再研究下select形式这种语句会怎么样。

mysql [localhost] {msandbox} (test) > select id,str_col from tbl_name where str_col="xxx" = "yyy";
+----+---------+
| id | str_col |
+----+---------+
|  1 | aaa     |
|  2 | aaa     |
|  3 | aaa     |
|  4 | aaa     |
+----+---------+

我们发现,这个SQL将str_col='aaa'的记录也查找出来了,为什么呢?
  1. mysql [localhost] {msandbox} (test) > warnings
  2. Show warnings enabled.
  3. mysql [localhost] {msandbox} (test) > explain extended select id,str_col from tbl_name where str_col="xxx" = "yyy"\G
  4. *************************** 1. row ***************************
  5.            id: 1
  6.   select_type: SIMPLE
  7.         table: tbl_name
  8.          type: index
  9. possible_keys: NULL
  10.           key: idx_str
  11.       key_len: 33
  12.           ref: NULL
  13.          rows: 4
  14.      filtered: 100.00
  15.         Extra: Using where; Using index
  16. 1 row in set, 1 warning (0.00 sec)

  17. Note (Code 1003): /* select#1 */ select `test`.`tbl_name`.`id` AS `id`,`test`.`tbl_name`.`str_col` AS `str_col` from `test`.`tbl_name` where ((`test`.`tbl_name`.`str_col` = 'xxx') = 'yyy')
复制代码
这里他把where条件转化成了
  1. ((`test`.`tbl_name`.`str_col` = 'xxx') = 'yyy')
复制代码

这个条件的首先判断str_col 和'xxx'是否相等,如果相等,那么里面括号的值为1,如果不相等,就是0

然后0或者1再和和'yyy'进行判断,由于等号一边是int,另外一边是字符串,两边都转化为float进行比较,这就是MySQL中隐式转换导致的查询结果出现错误。'yyy'转化为浮点型为0,0和0比较恒等于1
  1. mysql [localhost] {msandbox} (test) > select 'yyy'+0.0;
  2. +-----------+
  3. | 'yyy'+0.0 |
  4. +-----------+
  5. |         0 |
  6. +-----------+

  7. 1 row in set, 1 warning (0.00 sec)

  8. mysql [localhost] {msandbox} (test) > select 0=0;
  9. +-----+
  10. | 0=0 |
  11. +-----+
  12. |   1 |
  13. +-----+
  14. 1 row in set (0.00 sec)
复制代码


这样导致结果恒成立,也就是select语句等价于以下SQL

select id,str_col from tbl_name where 1=1;

将查询出所有的记录。

三、小结
在写SQL的过程中,一定要小心引号的位置是否正确,有时候引号位置错误,SQL依然是正常的,但是却会导致执行结果全部错误。在执行前必须在测试环境执行测试,结合IDE的语法高亮发现相应的问题。



















回复

使用道具 举报

您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

小黑屋|安全矩阵

GMT+8, 2024-11-28 17:54 , Processed in 0.013057 second(s), 18 queries .

Powered by Discuz! X4.0

Copyright © 2001-2020, Tencent Cloud.

快速回复 返回顶部 返回列表