?

Log in

No account? Create an account
Igor M Podlesny
xfs: = vs == typo in ASSERT() 
15th-Sep-2013 02:40 am
$ git show aa9e104
commit aa9e10409eae9dd61a336c6307d0a0b538063970
Author: Dan Carpenter <dan.carpenter@oracle.com>
Date:   Thu Sep 12 00:17:31 2013 +0300

    xfs: = vs == typo in ASSERT()
   
    There is a '=' vs '==' typo so the ASSERT()s are always true.
   
    Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
    Reviewed-by: Dave Chinner <dchinner@redhat.com>
    Reviewed-by: Eric Sandeen <sandeen@redhat.com>
    Signed-off-by: Ben Myers <bpm@sgi.com>

diff --git a/fs/xfs/xfs_bmap_btree.c b/fs/xfs/xfs_bmap_btree.c
index 531b020..bb8de8e 100644
--- a/fs/xfs/xfs_bmap_btree.c
+++ b/fs/xfs/xfs_bmap_btree.c
@@ -957,9 +957,9 @@ xfs_bmbt_change_owner(
        ASSERT(tp || buffer_list);
        ASSERT(!(tp && buffer_list));
        if (whichfork == XFS_DATA_FORK)
-               ASSERT(ip->i_d.di_format = XFS_DINODE_FMT_BTREE);
+               ASSERT(ip->i_d.di_format == XFS_DINODE_FMT_BTREE);
        else
-               ASSERT(ip->i_d.di_aformat = XFS_DINODE_FMT_BTREE);
+               ASSERT(ip->i_d.di_aformat == XFS_DINODE_FMT_BTREE);
 
        cur = xfs_bmbt_init_cursor(ip->i_mount, tp, ip, whichfork);
— ну заiбись, хорошо поассертились…
Comments 
14th-Sep-2013 11:05 pm (UTC)
Classic! Компиляторы же вроде бы давно об этом предупреждают.
15th-Sep-2013 08:12 am (UTC)
Ну… Большой беды, конечно, в этом нет, ибо если ожидается такое значение, то установка в такое же, разумеется, не должна повредить, но, опять же, разумеется, проверки в действительности нет.
27th-Dec-2016 01:38 am (UTC)
поэтому есть такой непривычный, неудобный, но разумный совет константу ставить слева
30th-Dec-2016 01:18 pm (UTC) - > есть такой непривычный
Есть такая книжка — «ален голуб: веревка достаточной длины чтобы выстрелить себе в ногу». Я прочёл её где-то в 1990-х. Кроме этого трюка там было ещё много чего неплохого.
This page was loaded Aug 19th 2019, 1:21 am GMT.