illumos gate

uploaded a webrev with the change:

http://cr.illumos.org/view/a8i2l5mn/

up to you to validate (and accept?) it...

Hi
You need not to work on usr/src/tools/env/illumos.sh. Please, make a private of this file and modify it.

Looks okay to me, now.
As I understand, you've built and installed it?

yes, have build and installed it (of course extremely minor change, so no impact and hard to simulate the issue)

Output from "hg pbchk" :

$ cat /tmp/hg-pbchk.out
Copyright check:

C style check:
Could not execute cstyle: [Errno 2] No such file or directory

Header format check:

Java style check:

Mapfile comment check:

File permission check:

Keywords check:

Comments check:

Checking for new tags:

Checking for multiple heads (or branches):

Checking for branch changes:

Checking for uncommitted changes:
Warning: the following files have uncommitted changes:
usr/src/cmd/su/su.c

Checking for merges:

Rob, I tried to contact you on IRC, but I guess you missed my message.

I already have this done:
http://pkgdev.openindiana.org/~trisk/old/il_cve-2010-3503/

...which is identical to your changes except for style guidelines.

If you want to pursue getting this put back, I can review it (and approve it once you get another reviewer). There is a testcase for the issue attached to the CVE iirc, which I used.

Albert,

yes please, review it (and approve, hopefully). If you can help me to identify other reviewers, that would be nice.

rgds
Rob

all,

please review my change:

(it has preliminary been reviewed already by Damian Wojslaw)

% hg pbchkk
Copyright check:

C style check:

Header format check:

Java style check:

Mapfile comment check:

File permission check:

Keywords check:

Comments check:
These IDs appear more than once in your comments:
  344

Checking for new tags:

Checking for multiple heads (or branches):

Checking for branch changes:

Checking for uncommitted changes:

Checking for merges:

% hg -v outgoing

running ssh anonhg@hg.illumos.org "hg -R illumos-gate serve --stdio" 
comparing with ssh://anonhg@hg.illumos.org/illumos-gate
searching for changes

changeset:   13246:c39379982564
user:        rob <rob@openindiana.local>
date:        Mon Dec 13 13:31:16 2010 +0100

description:
    344 su fails to check malloc's return value

modified:
   usr/src/cmd/su/su.c

changeset:   13247:22dfc1060a79
tag:         tip
user:        rob <rob@openindiana.local>
date:        Tue Dec 14 13:20:18 2010 +0100

description:
    344 su fails to check malloc's return value

modified:
   usr/src/cmd/su/su.c

% hg export tip
# HG changeset patch
# User rob <rob@openindiana.local>
# Date 1292329218 -3600
# Node ID 22dfc1060a79ef2d710209f97581ff377fa07d2f
# Parent  c393799825649271d04aab7eb8363164d626e45f
344 su fails to check malloc's return value

diff -r c39379982564 -r 22dfc1060a79 usr/src/cmd/su/su.c
--- a/usr/src/cmd/su/su.c    Mon Dec 13 13:31:16 2010 +0100
+++ b/usr/src/cmd/su/su.c    Tue Dec 14 13:20:18 2010 +0100
@@ -535,9 +535,10 @@
                         malloc(strlen(initenv[j])
                         + strlen(initvar)
                         + 2);
-                                        if (var == NULL) {
-                       perror("malloc"); exit(4);
-                                        }
+                    if (var == NULL) {
+                        perror("malloc");
+                        exit(4);
+                    }
                     (void) strcpy(var, initenv[j]);
                     (void) strcat(var, "=");
                     (void) strcat(var, initvar);

The diff is incorrect.

Can you update the patch to be against the correct tree? (It would likely be identical to the one I showed you).