+       printf(msg);

This really should be

        printf("%s", msg);

To avoid format string problems.

Yes, you say you are completely in control of the string.... however
someone could reuse this workq handler for some other purpose later.

Reply via email to