diff options
| author | Dan McGee <dan@archlinux.org> | 2008-12-29 22:36:07 -0600 |
|---|---|---|
| committer | Loui Chang <louipc.ist@gmail.com> | 2009-01-04 14:28:24 -0500 |
| commit | 207b565f1c05c556f05f4a549afeeb8878a3c372 (patch) | |
| tree | 0b49734a925b187c906f918f684bfa6292743d0a /web/html | |
| parent | 8f97330100af0baa89e635046bc97554782ec571 (diff) | |
| download | aurweb-207b565f1c05c556f05f4a549afeeb8878a3c372.tar.xz | |
Minimize calls to uid_from_sid()
Just like the previous patch for account_from_sid() over-usage.
Signed-off-by: Dan McGee <dan@archlinux.org>
Signed-off-by: Loui Chang <louipc.ist@gmail.com>
Diffstat (limited to 'web/html')
| -rw-r--r-- | web/html/pkgsubmit.php | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/web/html/pkgsubmit.php b/web/html/pkgsubmit.php index a7b4152..df25360 100644 --- a/web/html/pkgsubmit.php +++ b/web/html/pkgsubmit.php @@ -25,8 +25,10 @@ if ($_COOKIE["AURSID"]): $error = __("Error - No file uploaded"); } + $uid = uid_from_sid($_COOKIE['AURSID']); + # Temporary dir to put the tarball contents - $tempdir = UPLOAD_DIR . uid_from_sid($_COOKIE['AURSID']) . time(); + $tempdir = UPLOAD_DIR . $uid . time(); if (!$error) { if (!@mkdir($tempdir)) { @@ -278,8 +280,8 @@ if ($_COOKIE["AURSID"]): # If the package was a dummy, undummy it if ($pdata['DummyPkg']) { $q = sprintf( "UPDATE Packages SET DummyPkg = 0, SubmitterUID = %d, MaintainerUID = %d, SubmittedTS = UNIX_TIMESTAMP() WHERE ID = %d", - uid_from_sid($_COOKIE["AURSID"]), - uid_from_sid($_COOKIE["AURSID"]), + $uid, + $uid, $pdata["ID"]); db_query($q, $dbh); @@ -346,8 +348,8 @@ if ($_COOKIE["AURSID"]): mysql_real_escape_string($_REQUEST['category']), mysql_real_escape_string($new_pkgbuild['pkgdesc']), mysql_real_escape_string($new_pkgbuild['url']), - uid_from_sid($_COOKIE["AURSID"]), - uid_from_sid($_COOKIE["AURSID"]), + $uid, + $uid, mysql_real_escape_string($incoming_pkgdir . "/" . $pkg_name . ".tar.gz"), mysql_real_escape_string(URL_DIR . $pkg_name . "/" . $pkg_name . ".tar.gz")); |