[pve-devel] problem with my nexenta plugin when start vm with pve-manager, need perl help
Dietmar Maurer
dietmar at proxmox.com
Tue Aug 21 16:22:17 CEST 2012
Why you read something form external files, you need to parse the data.
That’s all - else perl taint mode throws errors.
There is no need to parse anything inside run_command.
- Dietmar
> -----Original Message-----
> From: pve-devel-bounces at pve.proxmox.com [mailto:pve-devel-
> bounces at pve.proxmox.com] On Behalf Of Alexandre DERUMIER
> Sent: Dienstag, 21. August 2012 15:01
> To: Stefan Priebe - Profihost AG
> Cc: pve-devel at pve.proxmox.com
> Subject: Re: [pve-devel] problem with my nexenta plugin when start vm with
> pve-manager, need perl help
>
> >>No problem but are there any other cases where we pass parameters to a
> >>system call? We need to regex check them all...
> What was the problem exactly ?
>
>
> I don't see param verification in
> /usr/share/perl5/PVE/Tools.pm
> sub run_command {
> my ($cmd, %param) = @_;
> ....
> $pid = open3($writer, $reader, $error, @$cmd) || die $!;
>
>
> ----- Mail original -----
>
> De: "Stefan Priebe - Profihost AG" <s.priebe at profihost.ag>
> À: "Alexandre DERUMIER" <aderumier at odiso.com>
> Cc: pve-devel at pve.proxmox.com
> Envoyé: Mardi 21 Août 2012 14:53:39
> Objet: Re: problem with my nexenta plugin when start vm with pve-manager,
> need perl help
>
> Am 21.08.2012 14:39, schrieb Alexandre DERUMIER:
> > Oh, It's working !
> >
> > Thanks !
> >
> > I'll send a patch.
>
> No problem but are there any other cases where we pass parameters to a
> system call? We need to regex check them all...
>
> Stefan
>
>
>
> --
>
> --
>
>
>
>
>
> Alexandre D e rumier
>
> Ingénieur Systèmes et Réseaux
>
>
> Fixe : 03 20 68 88 85
>
> Fax : 03 20 68 90 88
>
>
> 45 Bvd du Général Leclerc 59100 Roubaix
> 12 rue Marivaux 75002 Paris
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
More information about the pve-devel
mailing list