Re: [PATCH v3 01/30] syntax-check: sc_prohibit_nonreentrant: skip comments

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 20, 2022 at 21:07:50 +0200, Victor Toso wrote:
> It doesn't need to fail the test if a reentrant function is found in
> a comment.

That is true, but your patch doesn't do exactly that.

If I add the following code:

@@ -83,6 +83,12 @@ virshCatchDisconnect(virConnectPtr conn,
         const char *str = "unknown reason";
         virErrorPtr error;
         g_autofree char *uri = NULL;
+        long r;
+
+        r = random(); /* let's play a game */
+
+        if (r == 4)
+            abort();



It fails prior to this patch, but succeeds after it.

> 
> Fixes:
>  | ...
>  | prohibit_nonreentrant
>  | libvirt/ci/helper:33:            default=os.getlogin(),  # exempt from syntax-check
>  | libvirt/include/libvirt/libvirt-storage.h:223:    VIR_STORAGE_VOL_WIPE_ALG_PFITZNER7 = 6, /* 7-pass random (Since: v1.0.0) */
>  | libvirt/include/libvirt/libvirt-storage.h:225:    VIR_STORAGE_VOL_WIPE_ALG_PFITZNER33 = 7, /* 33-pass random (Since: v1.0.0) */
>  | libvirt/include/libvirt/libvirt-storage.h:227:    VIR_STORAGE_VOL_WIPE_ALG_RANDOM = 8, /* 1-pass random (Since: v1.0.0) */

Since it's very hard to parse C with regexes I suggest to change

 'pass random' to 'pass random data'

in the comments above to sidestep the issue.




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux