I'd like to point out that CGI scripts do not receive their arguments via @ARVG, so even if you fix if ($numArgs), your script still won't work. Look at CGI's param method.

Your script itself is extremely unsafe. The user could specify an arbitrary input or output directory name. But that's nothing compared to the user's ability to inject shell commands in $command.

Finally, why shell out to use ImageMagick when it provides a Perl interface? You're forking needlessly, and you're not even checking for success.

Your program needs much work.


In reply to Re: Use of uninitialized value in numeric lt (<) by ikegami
in thread Use of uninitialized value in numeric lt (<) by calypso

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":



  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.