Bug #1450: Illumos should be buildable with GCC4
undefined order of operations in kadmin GUI native code
cmd - userland programs
the kadmin GUI does this:
- for (i = 0; i < num_keysalts; grp[i] = i++);
The value of 'i' in the 3rd clause is undefined, there's no sequence point separating the use as an index from the increment.
We should instead do:
+ for (i = 0; i < num_keysalts; i++) + grp[i] = i;